Bugs in modified 1.06 r4356

[Jim]

@John_Delay: Kein Problem, mit den User-Namen kommt man leicht durcheinander.

So, noch ein paar Kleinigkeiten gefunden...

Beim bearbeiten die Language-Dateien ist mir folgendes aufgefallen:

lang/english/lang_english.conf, Zeilen 266-271, also das ganze Sektion

Code: XML  [Auswählen]

#newsletter
[account_newsletter]
heading_newsletter = 'Newsletter subscriptions.'
title_newsletter = 'My newsletter subscriptions'
text_newsletter = 'General newsletter'
text_newsletter_desc = 'Includes general news about our shop, information about new products, special offers and other information of general interest.

fehlt in lang/german/lang_german.conf

-----

lang/english/english.php

Zeile 152

Code: PHP  [Auswählen]

define('TEXT_RESULT_PAGE', 'Sites:');

besser wäre

Code: PHP  [Auswählen]

define('TEXT_RESULT_PAGE', 'Pages:');

Zeile 428

Code: PHP  [Auswählen]

define('ERROR_INVALID_USES_COUPON','This coucher can only be redeemed ');

coucher sollte voucher sein

Code: PHP  [Auswählen]

define('ERROR_INVALID_USES_COUPON','This voucher can only be redeemed ');

Schöne Grüße

Jim

[John_Delay]

Hallo Jim,

ja das kann ich bestätigen - könnte aber sein, dass dieser Block gar nicht benötigt wird momentan, oder?
Jedenfalls habe ich auch noch was gefunden. Sorry aber da musste ich mehr als schmunzeln: 
Und zwar die Konstanten
PAYPAL_INVOICE_TITLE  und
PAYPAL_INVOICE_DESC
in der lang/english/admin/configuration.php ... waia Jungs, das könnt ihr so nicht stehen lassen, auch wenn die Konstanten ggf. im Admin-Bereich derzeit noch nicht auftauchen. Wer hat das nur verzapft? 

Im Deutsch würde ich vielleicht besser schreiben:

Code: PHP  [Auswählen]

define('PAYPAL_INVOICE_TITLE','Shop-Präfix für PayPal Rechnungs-Nr.');
define('PAYPAL_INVOICE_DESC','Frei wählbare Buchstabenfolge (Präfix), die der Bestellnummer vorangestellt und für die Erzeugung der PayPal-Rechnungsnummer genutzt wird.<br />Dadurch k&ouml;nnen mehrere Shops mit nur einem PayPal Konto arbeiten. Konflikte bei gleichen Bestellnummern werden vermieden. Jede Bestellung erhält eine eigene Rechnungs-Nummern in dem PayPal Konto.');

Dann versteht man das auch *gg. Bin zwar kein Muttersprachler, aber ich mache mal für den Englischen Teil ´nen Alternativ-Vorschlag:

Code: PHP  [Auswählen]

define('PAYPAL_INVOICE_TITLE','Shop prefix for PayPal invoice no.');
define('PAYPAL_INVOICE_DESC','Arbitrary string of letters (prefix), which is placed in front of each order number and is used for generating the PayPal invoice number.<br />This allows multiple store operation with only one PayPal account. Conflicts regarding the order numbers are avoided. Each order has its own invoice numbers within the PayPal account.');

Gruß, Ingo

P.S.: Kurze, prägnante Sätze lassen sich in Google besser übersetzen. 

Ach und: die Konstante 'MODULE_PAYMENT_MASTERPAYMENT_CONFIG_SORT_ORDER_DESC' fehlt in der englischen lang/english/modules/payment/masterpayment_config.php gänzlich

[John_Delay]

zu meinem letzten P.S.:
Statt dessen steht dort MODULE_PAYMENT_MASTERPAYMENT_CONFIG_CARD_SORT_ORDER_DESC ... das 'CARD_' ist dort zu viel.

Und dann habe ich noch einen Vorschlag: @theTeam

Habe mich immer darüber geärgert, dass die Radio-Buttons (z. B. im Registrierungsformular) so dicht an 'Frau/Herr' stehen. Um das in zukünftigen Versionen zu ändern, sucht in den Dateien:

create_account.php
create_guest_account.php
includes/modules/checkout_new_address.php

nach den Konstanten INPUT_FEMALE und INPUT_MALE .
In den betreffenen Zeilen (je 2 pro Datei) fügt Ihr dann hinter dem FEMALE bzw. MALE noch ein Leerzeichen ein mit .'&nbsp;' - also beispielsweise so:

Code: PHP  [Auswählen]

... xtc_draw_radio_field(array ('name' => 'gender', 'suffix' => MALE.'&nbsp;' ...

In den Dateien

account_edit.php
includes/modules/address_book_details.php

wurde genau das bereits zu einem früheren Zeitpunkt geändert. Diese müssen also nicht mehr bearbeitet werden. Jetzt könnte einer sagen: "Ändere doch einfach die Sprachkonstanten FEMALE und MALE und setze ein Leerzeichen dahinter" (also 'Herr ' statt 'Herr' etc.) ... Aber Pustekuchen *gg, denn dann funzt das mit der automatischen (geschlechtsspezifischen) Anrede in den (Bestell-)Emails nicht mehr bzw. funzt es nur dann, wenn man es in den Email.html- und Email.txt-Dateien auch ändert.

Gruß, Ingo

[John_Delay]

Und wo wir schon mal dabei sind: 

Wenn ich im Demo-Shop alle Zahlungsmodule installiere, eine Reihenfolge und alle auf 'true' setze, erscheinen im Frontend nur diese hier:

DIRECTebanking.com   
PayPal Checkout   
Giropay   
Credit Card   
Sofortueberweisung

hmmmm, ok, das Paypal etc. ohne Registrierung nicht geht ist klar ... aber warum nur diese 5? Kann man irgendwo die maximale Anzahl begrenzen oder stimmt da was nicht? Vlt. bin ich auch einfach nur zu müde ...

Gn8 & Gruß, Ingo

P.S.: Liest das hier überhaupt wer? 

[ralph_84]

@ Team und @Ingo

Ja das kann ich nur bestätigen, ist bei mir in der 1.06 auch so leider.
Es werden nur 5 Rechnungsarten angezeigt.

Aber habe noch einen Fehler gefunden!
Wenn eine Bestellung gelöscht wird, dann kann ich ja sagen,
den Artikel dem Lager Gutschreiben.
Doch leider wird der Artikel nicht mehr dem Lager zugefügt!
Warum ist das so? In der 1.05 Sp1d geht es aber noch.

Gruß Ralph

[paco43]

Wann kommt denn ein neues update wo die ganzen Fehler weg sind?

Gruß Paco

[noRiddle (revilonetz)]

"...wo die ganzen Fehler weg sind..."
Wie kommt es nur daß mir immer öfter dieses eine Zitat von dem nicht unbedingt von mir geschätzten Atze Schröder einfällt:
"Ja neee, iss klaaar..."

noRiddle

[paco43]

 

[Dato]

Wann kommt denn ein neues update wo die ganzen Fehler weg sind?

Das würde ich mich auch mal Interessieren, denn es sind 3 Seiten Fehler gepostet worden.

[jannemann]

Das Problem, dass nicht alle installierten Zahlungsmodule angezeigt werden, wurde in r4530 gefixt.

Schöne Grüße,
Jan

[lohkaes1]

Hallo,

wenn ich im Admin die Zahlungsart "Vorkasse/Überweisung" ausgewählt habe, zeigt mir die checkout_process.php nur eine leere Seite an.

Hab mal in der checkout_process.php nach der Stelle gesucht(Zeile ca. 150):

Code: PHP  [Auswählen]

// check if tmp order id exists
if (isset ($_SESSION['tmp_oID']) && is_numeric($_SESSION['tmp_oID'])) { // Dokuman - 2009-10-11 - Paypal fix for infinite loop see, http://www.modified-shop.org/forum/topic.php?id=2235
  $tmp = false;
  $insert_id = $_SESSION['tmp_oID'];
} else {
  // check if tmp order need to be created
  if (isset ($$_SESSION['payment']->form_action_url) && $$_SESSION['payment']->tmpOrders) {
    $tmp = true;
    $tmp_status = $$_SESSION['payment']->tmpStatus;
  } else {
    $tmp = false;
    $tmp_status = $order->info['order_status'];
  }

Die Variable $tmp_status scheint leer zu sein.

wo liegt hier das Problem? Der Checkout mit Paypal geht.

Danke.

[jannemann]

Hi,

kannst du das Problem im Demoshop nachstellen?

Schöne Grüße,
Jan

[lohkaes1]

Hi Jan,

wollte ich gerade versuchen und erhalte die Phishing-Warnung von Google!!!

Gruß

[jannemann]

Die erhalte ich auch. Liegt am Browser.

[lohkaes1]

Hallo Jan,

hab das jetzt im DemoShop versucht und bekomme hier keine leere-Seite. Hast Du eventuell eine Idee wo ich da anpacken  muss?

Gruß