Helft uns die kommende Version 1.03 zu testen!

[TomPH]

Und ich mal wieder. Muss ja die Anzahl meiner Post's etwas hochjagen

Ich habe ein Problem gefunden. Wenn ich bei den Artikeln Staffelpreise eingebe, werde ich nach dem Speichern jedes Staffelpreises wieder aus dem Artikel geworfen und ende in der Artikel-Listung. Etwas stressig, wenn ich 10 Staffelpreise brauche.

Viele Grüsse von den Philippinen (31 Grad und Sonne)
Tom

[Tomcraft]

Das ist ein generelles xt:Commerce-Problem, was erstmal nicht für 1.03 gefixt wird.

Grüße

Torsten

[vr]

Hallo,

bin neu hier. Habe gleich mal 1.03 getestet.

1. Was Benny in http://www.modified-shop.org/forum/topic.php?id=245#post-1304 vorgeschlagen hat, steht immer noch aus, die erweiterte Suche berücksichtigt nicht die USt, dh suche ich als Normalkunde nach Artikeln bis 30E, kriege ich auch die für 33.95E angezeigt.

2. Die Sortierung der Trefferliste geht vermutlich immer noch nach products_id. Das bringt Anwendern nix. Stattdessen könnte bspw nach products_name sortiert ausgegeben werden.

Grüße Volker

[vr]

Re 1.03: Gibts ein testlogin für den Adminbereich?

Grüße, Volker

[vr]

Hi,

weiter Testergebnisse:
erweiterte Suche:
1. Wenn nichts gefunden wird, wird über dem Hinweis die Standardsuche eingeblendet. War schon immer so, ist aber unpraktisch.

2. Bei Suche über 'Preis bis': Gibt man 30,50 an, kriegt man die korrekte Treffermenge (bis auf den Steueranteil, s. früheres Posting). Gibt man 0,50 an, kriegt man *alle* verfügbaren Artikel. Die Suche nach 30.50 und 0.50 verhält sich hingegen korrekt.

Könnt ihr zum Testen bitte auch Artikel mit 7% USt einstellen?

Danke,
Grüße, vr

[vr]

Hi,

Thema Bankdatenvalidierung:

Die Datei banktransfer_validation.php (kenne nur die von Version 1.02) enthält einige, zT gravierende Bugs, ausserdem fehlen die neu hinzugekommenen Prüfmethoden C1 bis D3. Das führt dazu, dass in bestimmten Fällen Bankdaten abgelehnt werden, obwohl sie korrekt sind.

Habt ihr Interesse an einer Version, wo die Bugs, die mir aufgefallen sind, behoben sind, und die fehlenden Prüfmethoden implementiert sind?

Die amtliche Seite zum Thema Bankdatenvalidierung ist die hier:

http://www.bundesbank.de/zahlungsverkehr/zahlungsverkehr_bankleitzahlen_download.php

dort gibt es die aktuellen Bankleitzahlen und zugehörigen Prüfmethoden (die ändern sich alle paar Monate geringfügig) zum Download. Ich hab die Bundesbank mal angeschrieben, ob sie die Dateien nicht zum automatischen Download anbieten, zb in dem sie die aktuelle Datei immer gleich benennen, damit sich ein shop die selber abholen kann, aber davon wollten sie nix wissen. Also muss man, wenn man als Shopbetreiber bankdaten checken will, alle 2-3 Monate auf die obige Bundesbankseite und sich dort (downloadlinks im unteren Teil der Seite) die aktuelle blz-Tabelle abholen. Die muss man dann in die Tabelle banktransfer_blz einspielen.

(Es wäre vor dem Hintergrund natürlich ein cooler Service für modified eCommerce Shopsoftware-Anwender, wenn die aktuelle blz-Datei (aufs wesentliche reduziert) auf dem Server von modified eCommerce Shopsoftware unter immer dem gleichen Namen abrufbar wäre. Dann könnte sich ein shop bspw über wddx die Daten ziehen und sich aktualisieren. So wie es derzeit von der Bundesbank gehandhabt wird, ist es hingegen eine unnötige und nervige ABM für jeden shop-Admin, dessen shop Bankdaten validieren soll)

Die Definitionen für die Prüfmethoden in banktransfer_validation.php stehen auf der Seite
http://www.bundesbank.de/zahlungsverkehr/zahlungsverkehr_pruefziffernberechnung.php
und da in einem link namens Prüfzifferberechnungsmethode, was ein pdf ist. Da sich auch da der Namen regelmäßig ändert, kann ich den hier nicht als link angeben.

Grüße, Volker

[Tomcraft]

Hi Volker,

schau mal hier:

Das Problem wurde erst vor 2 Wochen gefixt.
Aktuelle Dateien, siehe: https://trac.modified-shop.org/changeset/488/

Grüße

Torsten

[vr]

Hi Volker,

schau mal hier:

Sehr schön, danke für den Hinweis. Das sieht richtig knackig aus! Folgendes ist mir aufgefallen:

1. In der xtc-Anleitung v2.10.pdf wird unter Punkt 11.20 noch die falsche Abfrage als Ersetzung genannt:

statt

$blz_query = xtc_db_query("SELECT * from banktransfer WHERE banktransfer_blz = '" . $blz . "'");

muss es heissen

$blz_query = xtc_db_query("SELECT * from banktransfer_blz WHERE blz = '" . $blz . "'");

2. Mark75 braucht die 0-getrimmte Länge der Kontonummer, bei expandierten Nummern funktioniert das case sonst nicht, da die immer die Länge 10 haben. Also $account['len'] statt strlen($AccountNo). Ausserdem ist bei den letzten beiden Aufrufen der Method00 die Significance invertiert. Statt

$Help = $this->Method00($AccountNo,'012121',7);

muss es heissen

$Help = $this->Method00($AccountNo,'021212', 7);

3. In Funktion CheckAccount werden einige Methoden aufgerufen, die BLZ als zusätzlichen Parameter brauchen. Hier fehlt der Eintrag für Prüfmethode C7:

case "C7" : $Result = $this->MarkC7($KontoNR, $adata['blz']); break;

4. Wärs eine Idee, den Rückgabecode 2 (Für diese Kontonummer ist kein Prüfziffernverfahren definiert) auf 0 (BLZ und Kto OK) abzubilden? Vorn im shop können Anwender mit der Fehlermeldung zu Retcode 2 nichts anfangen, sie können ja nichts dran ändern, ihre Bankdaten sind korrekt, werden aber trotzdem nicht akzeptiert. Oder sollte das eher an den Aufrufstellen der Klasse behandelt werden?

Grüße, Volker

[DokuMan]

Hallo Volker,

danke für die Arbeit, die du dir gemacht hast!
1. Doku aktualisiert (kommt mit 2.11)!
2. wurde eben in banktransfer_validation.php implementiert (@540)
3. wurde eben in banktransfer_validation.php implementiert (@540)
4. das könnte man machen! Wäre eine gute Idee!

Darüberhinaus bräuchten wir für das banktransfer-Modul noch einen Schalter, die BLZ-Prüfung komplett zu deaktivieren! Wie es scheint, greift sie immer!
(siehe http://www.modified-shop.org/forum/topic.php?id=2646#post-24788 )

[vr]

danke für die Arbeit, die du dir gemacht hast!
1. Doku aktualisiert (kommt mit 2.11)!

Gerne, und danke gleichfalls. Endlich eine community

2. wurde eben in banktransfer_validation.php implementiert (@540)

nicht ganz, die Verwendung von $account['len'] statt strlen($AccountNo) bei Mark75 fehlt noch.

4. das könnte man machen! Wäre eine gute Idee!

Darüberhinaus bräuchten wir für das banktransfer-Modul noch einen Schalter, die BLZ-Prüfung komplett zu deaktivieren! Wie es scheint, greift sie immer!
(siehe http://www.modified-shop.org/forum/topic.php?id=2646#post-24788 )

In banktransfer.php wird ja u.a. nur dann validiert, wenn $_POST['banktransfer_fax'] == false. Nützt aber nix, weil man nicht immer statt Bankdatencheck faxen will . Am ehesten sehe ich die Abschaltlogik ganz am Anfang von CheckAccount, gesteuert durch einen neuen Schalter analog MODULE_PAYMENT_BANKTRANSFER_DATABASE_BLZ, zb so:

Code: PHP  [Auswählen]

function CheckAccount($banktransfer_number, $banktransfer_blz) {
  if (MODULE_PAYMENT_BANKTRANSFER_VALIDATION == false)
    return 0;
  ...
 

Grüße, Volker

ps: wie kann man hier eigentlich inline zitieren?

[vr]

danke für die Arbeit, die du dir gemacht hast!
1. Doku aktualisiert (kommt mir 2.11)!

Gerne, und danke gleichfalls. Endlich eine community

2. wurde eben in banktransfer_validation.php implementiert (@540)

nicht ganz, die Verwendung von $account['len'] statt strlen($AccountNo) bei Mark75 fehlt noch.

4. das könnte man machen! Wäre eine gute Idee!

Darüberhinaus bräuchten wir für das banktransfer-Modul noch einen Schalter, die BLZ-Prüfung komplett zu deaktivieren! Wie es scheint, greift sie immer!
(siehe http://www.modified-shop.org/forum/topic.php?id=2646#post-24788 )

In banktransfer.php wird ja u.a. nur dann validiert, wenn $_POST['banktransfer_fax'] == false. Nützt aber nix, weil man nicht immer statt Bankdatencheck faxen will . Am ehesten sehe ich die Abschaltlogik ganz am Anfang von CheckAccount, gesteuert durch einen neuen Schalter analog MODULE_PAYMENT_BANKTRANSFER_DATABASE_BLZ, zb so:

Code: PHP  [Auswählen]

function CheckAccount($banktransfer_number, $banktransfer_blz) {
  if (MODULE_PAYMENT_BANKTRANSFER_VALIDATION == false)
    return 0;
  ...
 

Grüße, Volker

ps: wie kann man hier eigentlich inline zitieren?

[vr]

danke für die Arbeit, die du dir gemacht hast!
1. Doku aktualisiert (kommt mir 2.11)!

Gerne, und danke gleichfalls. Endlich eine community

2. wurde eben in banktransfer_validation.php implementiert (@540)

nicht ganz, die Verwendung von $account['len'] statt strlen($AccountNo) bei Mark75 fehlt noch.

4. das könnte man machen! Wäre eine gute Idee!

Darüberhinaus bräuchten wir für das banktransfer-Modul noch einen Schalter, die BLZ-Prüfung komplett zu deaktivieren! Wie es scheint, greift sie immer!
(siehe http://www.modified-shop.org/forum/topic.php?id=2646#post-24788 )

In banktransfer.php wird ja u.a. nur dann validiert, wenn $_POST['banktransfer_fax'] == false. Nützt aber nix, weil man nicht immer statt Bankdatencheck faxen will . Am ehesten sehe ich die Abschaltlogik ganz am Anfang von CheckAccount, gesteuert durch einen neuen Schalter analog MODULE_PAYMENT_BANKTRANSFER_DATABASE_BLZ, zb so:

Code: PHP  [Auswählen]

function CheckAccount($banktransfer_number, $banktransfer_blz) {
  if (MODULE_PAYMENT_BANKTRANSFER_VALIDATION == false)
    return 0;
  ...
 

Grüße, Volker

ps: wie kann man hier eigentlich inline zitieren?

[vr]

danke für die Arbeit, die du dir gemacht hast!
1. Doku aktualisiert (kommt mir 2.11)!

Gerne, und danke gleichfalls. Endlich eine community

2. wurde eben in banktransfer_validation.php implementiert (@540)

nicht ganz, die Verwendung von $account['len'] statt strlen($AccountNo) bei Mark75 fehlt noch.

4. das könnte man machen! Wäre eine gute Idee!

Darüberhinaus bräuchten wir für das banktransfer-Modul noch einen Schalter, die BLZ-Prüfung komplett zu deaktivieren! Wie es scheint, greift sie immer!
(siehe http://www.modified-shop.org/forum/topic.php?id=2646#post-24788 )

In banktransfer.php wird ja u.a. nur dann validiert, wenn $_POST['banktransfer_fax'] == false. Nützt aber nix, weil man nicht immer statt Bankdatencheck faxen will . Am ehesten sehe ich die Abschaltlogik ganz am Anfang von CheckAccount, gesteuert durch einen neuen Schalter analog MODULE_PAYMENT_BANKTRANSFER_DATABASE_BLZ, zb so:

Code: PHP  [Auswählen]

function CheckAccount($banktransfer_number, $banktransfer_blz) {
  if (MODULE_PAYMENT_BANKTRANSFER_VALIDATION == false)
    return 0;
  ...
 

Grüße, Volker

ps: wie kann man hier eigentlich inline zitieren?

[vr]

Hallo Volker,

danke für die Arbeit, die du dir gemacht hast!
1. Doku aktualisiert (kommt mir 2.11)!
2. wurde eben in banktransfer_validation.php implementiert (@540)
3. wurde eben in banktransfer_validation.php implementiert (@540)
4. das könnte man machen! Wäre eine gute Idee!

Darüberhinaus bräuchten wir für das banktransfer-Modul noch einen Schalter, die BLZ-Prüfung komplett zu deaktivieren! Wie es scheint, greift sie immer!
(siehe http://www.modified-shop.org/forum/topic.php?id=2646#post-24788 )

Zu 1.: Gerne, und danke gleichfalls. Endlich eine community

Zu 3.: nicht ganz, die Verwendung von $account['len'] statt strlen($AccountNo) bei Mark75 fehlt noch.

In banktransfer.php wird ja u.a. nur dann validiert, wenn $_POST['banktransfer_fax'] == false. Nützt aber nix, weil man nicht immer statt Bankdatencheck faxen will . Am ehesten sehe ich die Abschaltlogik ganz am Anfang von CheckAccount, gesteuert durch einen neuen Schalter analog MODULE_PAYMENT_BANKTRANSFER_DATABASE_BLZ, zb so:

Code: PHP  [Auswählen]

function CheckAccount($banktransfer_number, $banktransfer_blz) {
  if (MODULE_PAYMENT_BANKTRANSFER_VALIDATION == false)
    return 0;
  ...
 

Grüße, Volker

ps: wie kann man hier eigentlich inline zitieren?

[vr]

Hallo Volker,

danke für die Arbeit, die du dir gemacht hast!

Gerne, und danke gleichfalls. Endlich eine community

(Komisch, ich hab versucht, Dein letztes Posting am Stück zu beantworten, die ForenSW akzeptiert aber keine Texte mehr von mir. Ich versuchs mal in kleinen Happen.)

Grüße Volker