Bugs in modified 1.06 r4356

[usernamenotfound]

Danke!

[GTB]

das ist aber nicht so einfach zu lösen.

In der Übersicht, wird die Kaufsumme angezeigt. Wenn du die ot_total entfernt hast und nur mit ot_subtotal arbeitest, dann muss man dazu folgende Änderungen machen.

/admin/orders.php

suche das hier:

Code: PHP  [Auswählen]

                    } elseif (isset($_GET['status']) && $_GET['status']=='0') {
                        $orders_query_raw = "-- /admin/orders.php
                                             SELECT ".$order_select_fields."
                                               FROM ".TABLE_ORDERS." o
                                          LEFT JOIN ".TABLE_ORDERS_TOTAL." ot ON (o.orders_id = ot.orders_id)
                                               WHERE o.orders_status = '0'
                                                 AND ot.class = 'ot_total'
                                            ORDER BY o.orders_id DESC";

                    } elseif (isset($_GET['status']) && xtc_not_null($_GET['status'])) { //web28 - 2012-04-14  - FIX xtc_not_null($_GET['status'])
                        $status = xtc_db_prepare_input($_GET['status']);
                        $orders_query_raw = "-- /admin/orders.php
                                             SELECT ".$order_select_fields.",
                                                    s.orders_status_name
                                               FROM ".TABLE_ORDERS." o
                                          LEFT JOIN (".TABLE_ORDERS_TOTAL." ot, ".TABLE_ORDERS_STATUS." s)
                                                 ON (o.orders_id = ot.orders_id AND o.orders_status = s.orders_status_id)
                                               WHERE s.language_id = '".(int)$_SESSION['languages_id']."'
                                                 AND s.orders_status_id = '".xtc_db_input($status)."'
                                                 AND ot.class = 'ot_total'
                                            ORDER BY o.orders_id DESC";

                    } elseif ($action == 'search' && $oID) {
                         // ADMIN SEARCH BAR $orders_query_raw moved it to the top
                    } else {
                          $orders_query_raw = "-- /admin/orders.php
                                               SELECT ".$order_select_fields.",
                                                      s.orders_status_name
                                                 FROM ".TABLE_ORDERS." o
                                            LEFT JOIN (".TABLE_ORDERS_TOTAL." ot, ".TABLE_ORDERS_STATUS." s)
                                                   ON (o.orders_id = ot.orders_id AND o.orders_status = s.orders_status_id)
                                                WHERE (s.language_id = '".(int)$_SESSION['languages_id']."'
                                                        AND ot.class = 'ot_total')
                                                   OR (o.orders_status = '0'
                                                        AND ot.class = 'ot_total'
                                                        AND s.orders_status_id = '1'
                                                        AND s.language_id = '".(int)$_SESSION['languages_id']."')
                                             ORDER BY o.orders_id DESC";
 

die darin vorkommenden ot_total ersetzen mit ot_subtotal

Gruss Gerhard

[Tomcraft]

Ich bin mir ziemlich sicher, dass die Auflistung der Bestellungen in früheren Versionen nicht an das Vorhandensein von "ot_total" gebunden war.
Das sollten wir nochmal überarbeiten.

Grüße

Torsten

[GTB]

ich habe es gerade in der Version 1.00 getestet. Auch dort ist die gleiche Situation. Wenn ot_total deaktiviert wird, dann wird die Bestellung zwar in der Adminbox angezeigt, aber nicht im Admin in den Bestellungen.

Aber mir fällt da bestimmt was ein, wie man das lösen kann.

Gruss Gerhard

[GTB]

Hi,

so der Fehler ist gefixt in r4819

Gruss Gerhard

[Modulfux]

Nur wenn der Trac nicht mehr öffentlich ist, nützen einem die Fixes auch nichts.

Gruß
Ronny

[GTB]

Nur wenn der Trac nicht mehr öffentlich ist, nützen einem die Fixes auch nichts.

doch, ihr wisst, dass das Problem behoben ist und in der nächsten Version enthalten ist.

Gruss Gerhard

[Modulfux]

Ok, dann wäre es aber vermessen, insbesondere im Sinne der Community, demnächst User abzuwatschen, die ihre Lösungen nicht posten.

All zu oft liest man hier, wenn User ihre Lösungen für sich behalten, dass es doch angemessen wäre, diese zu präsentieren, falls noch andere User das selbe Problem haben.

Soll nicht mein Bier sein, aber ein wenig strange finde ich die neue Vorgehensweise schon.

Gruß
Ronny

[web0null]

@Webkiste 

@GTB,

doch, ihr wisst, dass das Problem behoben ist und in der nächsten Version enthalten ist.

Na wenn jetzt alle hier so denken, kann man das Forum ja schließen, und nur noch mehr die Updates bereitstellen, wenn sie fertig sind

Ist zwar glaube ich nicht wirklich der Sinn dieses Forums was ich nebenbei bemerkt, als eines der besten, schätze, die es überhaupt gibt. Aber naja ihr werdet schon eure gründe haben, jetzt einen neuen Weg einzuschlagen.

Ich meine bei eventuellen neuen Funktionen, die einen Mehrwert bedeuten, würde ich es ja verstehen, es nicht gleich öffentlich zu machen, aber bei fehlern??

... und es geht nicht darum das der Trac nicht mehr einsehbar ist sondern er geht um das kommunizieren der Lösung.

... und wenn der Trac gesperrt ist, dann bitte entfernt doch auch die beiden Download-Links unten, die ja leer sind.

In diesem Sinne beste Grüße web0null.

[John_Delay]

Hallo an das Team, hallo Ronny, hallo web0null,

kann mich der Meinung von Ronny und web0null nur anschließen. Auch, dass es bislang ein ausgesprochen gutes Forum war. Ein Forum sollte m. E. auch dafür da sein, alle (auch die von Seiten des Teams) gefixten Bugs zu testen und zu diskutieren, damit man ggf. nochmal ein Feedback geben kann, ob der Fix so ok ist. 100 Augen sehen mehr als 2, 4, 6 oder 8 ...

Aber mir fällt da bestimmt was ein, wie man das lösen kann.

@GTB: Hmmm, wenn Du zumindest HIER geschrieben hättest, was Dir anschließend eingefallen ist und was Du gefixt hast, hätte man es an der eigenen Seite nochmal prüfen können. Könnt Ihr Euch denn stets sicher sein, dass Ihr es KORREKT gefixt habt und daraus nicht ggf. Probleme an anderer Stelle entstanden sind bzw. alle Eventualitäten (unterschiedliche Shop-Konfigurationen) berücksichtigt wurden?

Wenn alle User (inkl. der Beta-Tester, die es jetzt ja nicht mehr gibt) erst mit dem neuen Release alle Fixes erhalten, sind die möglicherweise in den Fixes enthaltenen "neuen" Bugs nicht mehr so leicht auffindbar ... und aufgrund der Prozeduralität des Codes lassen sich Unit-Tests leider nicht so leicht implementieren. Die Beta-Tester sind da Eure einzige Rettung und leisten wahrsch. mehr Arbeit, als Euch derzeit bewusst ist. Die Zeit, die Ihr (das Team) jetzt spart, indem Ihr weniger zu posten habt, kommt daher ggf. bei Herausgabe des nächsten Releases als Bummerang doppelt oder dreifach zurück.

Wie bei den anderen schon durchklang: verstehen tut es niemand, ... aber es ist Eure Entscheidung und die haben wir zu akzeptieren.

Gruß, Ingo

[noRiddle (revilonetz)]

Prinzipiell verstehe ich auch nicht warum überhaupt noch mit changeset-Link informiert wird, was bringt der Link ?
Wie auch immer, ich finde das Problem als Fehler zu bezeichnen auch nicht korrekt.
order_total ist ja normalerweise essentiell für eine Bestellung, das zu deaktivieren ist doch eher eine Sonderanwendung, oder nicht ?

Gruß,
noRiddle

[GTB]

Ich habe bereits hier eine mögliche Lösung beschrieben.

Der Fall ist hier so speziell, dass das bisher seit Version 1.00 noch niemand bemerkt hat bzw das Problem hatte.

Die nächste Version kommt bestimmt und testen könnt ihr jederzeit im Demoshop

Gruss Gerhard

[GTB]

Prinzipiell verstehe ich auch nicht warum überhaupt noch mit changeset-Link informiert wird, was bringt der Link ?

der ist für uns wichtig

Wenn der Bugtracker richtig ausgereift ist und wir hier einen optimalen Arbeitsablauf haben, dann kann das überflüssig werden. Auch hier verweisen wir bereits auch Changesets.

Gruss Gerhard

[lithoverlag]

Kurze Mitteilung.
Es geht um den FCK Editor bei IE 10  (Andang des Threads). Einem Kunden, dem ich die (so dachte ich) neueste Version installiert habe konnte keinen Editor bei seinen Artikel im IE sehen.
Habe das Bugfix drüber genudelt und alles gut.
Das Bugfix scheint aber nicht in der neuesten Version implementiert zu sein.

[web28]

Danke für die Rückmeldung.

Meinst Du diesen Fix?

http://www.modified-shop.org/forum/index.php?topic=26188.msg239164#msg239164

Gruss Web28