Skip to content

Commit 05ba6f1

Browse files
committed
OX6-47: Added reference type change to be compatible with alphanumeric prefix
1 parent 3ce1cfe commit 05ba6f1

File tree

4 files changed

+9
-19
lines changed

4 files changed

+9
-19
lines changed

application/controllers/admin/fcpayone_main.php

+1-13
Original file line numberDiff line numberDiff line change
@@ -411,19 +411,7 @@ protected function _fcpoValidateData()
411411
*/
412412
protected function _fcpoValidateAccountData()
413413
{
414-
$aConfStrs = $this->_oFcpoHelper->fcpoGetRequestParameter("confstrs");
415-
416-
$blValidPrefix = is_numeric($aConfStrs['sFCPORefPrefix']);
417-
418-
if (!$blValidPrefix) {
419-
$this->_fcpoAddConfigError('FCPO_CONFERROR_PREFIX_NUMERIC');
420-
}
421-
422-
$blValid = (
423-
$blValidPrefix
424-
);
425-
426-
return $blValid;
414+
return true;
427415
}
428416

429417
/**

application/views/admin/de/fcPayOne_lang.php

-1
Original file line numberDiff line numberDiff line change
@@ -483,7 +483,6 @@
483483
'FCPO_EMAIL_IBAN' => 'IBAN:',
484484
'FCPO_EMAIL_CLEARING_BODY_THANKYOU' => 'Vielen Dank, dein %SHOPNAME%-Team',
485485
'FCPO_EMAIL_USAGE' => 'Verwendungszweck',
486-
'FCPO_CONFERROR_PREFIX_NUMERIC' => 'Der optionale Referenznummernpräfix muss numerisch sein.',
487486
);
488487

489488
/*

application/views/admin/en/fcPayOne_lang.php

-2
Original file line numberDiff line numberDiff line change
@@ -492,8 +492,6 @@
492492
'FCPO_EMAIL_IBAN' => 'IBAN:',
493493
'FCPO_EMAIL_CLEARING_BODY_THANKYOU' => 'Thank you, your %SHOPNAME%-Team',
494494
'FCPO_EMAIL_USAGE' => 'Usage',
495-
'FCPO_CONFERROR_PREFIX_NUMERIC' => 'The optional reference prefix must be numeric.',
496-
497495
);
498496

499497
/*

core/fcpayone_events.php

+8-3
Original file line numberDiff line numberDiff line change
@@ -550,7 +550,7 @@ public static function addDatabaseStructure()
550550

551551
self::changeColumnTypeIfWrong('fcpotransactionstatus', 'FCPO_USERID', 'varchar(32)', self::$sQueryChangeToVarchar1);
552552
self::changeColumnTypeIfWrong('fcpotransactionstatus', 'FCPO_TXID', 'varchar(32)', self::$sQueryChangeToVarchar2);
553-
self::changeColumnTypeIfWrong('fcporequestlog', 'FCPO_REFNR', 'varchar(32)', self::$sQueryChangeFcporequestlog);
553+
self::changeColumnTypeIfWrong('fcporequestlog', 'FCPO_REFNR', 'int(11)', self::$sQueryChangeFcporequestlog);
554554
self::changeColumnTypeIfWrong('oxorder', 'FCPOREFNR', 'varchar(32)', self::$sQueryChangeRefNrToVarchar);
555555

556556
self::dropIndexIfExists('fcporefnr', 'FCPO_REFNR');
@@ -642,9 +642,14 @@ public static function insertRowIfNotExists($sTableName, $aKeyValue, $sQuery)
642642
*/
643643
public static function changeColumnTypeIfWrong($sTableName, $sColumnName, $sExpectedType, $sQuery)
644644
{
645-
if (oxDb::getDb()->getOne("SHOW COLUMNS FROM {$sTableName} WHERE FIELD = '{$sColumnName}' AND TYPE = '{$sExpectedType}'")) {
645+
$sCheckQuery = "
646+
SHOW COLUMNS
647+
FROM {$sTableName}
648+
WHERE FIELD = '{$sColumnName}'
649+
AND TYPE = '{$sExpectedType}'
650+
";
651+
if (oxDb::getDb()->getOne($sCheckQuery)) {
646652
oxDb::getDb()->Execute($sQuery);
647-
// echo 'In Tabelle '.$sTableName.' Spalte '.$sColumnName.' auf Typ '.$sExpectedType.' umgestellt.<br>';
648653
return true;
649654
}
650655
return false;

0 commit comments

Comments
 (0)