Modify

Opened 9 years ago

Closed 9 years ago

#996 closed Bug/Fehler (fixed)

Fehlerhafte Anzeige in Steuerzonen

Reported by: arno@… Owned by: somebody
Priority: normal Milestone: modified-shop-2.0.2.0
Component: Admin Version: 2.0.0.0
Keywords: Cc:
Blocked By: Blocking:

Description

Durch einen "left join" im Query werden fehlende Länder als "Alle Länder" angezeigt.

Nun ist es zwar dumm, Länder zu löschen, statt sie zu deaktivieren. Der Fehler sollte also bei korrekter Bedienung nicht auftauchen. Unschön ist's dennoch.

Der Fehler tritt sowohl bei 1.06 bis SP4 auf, als auch bei 2.0

Attachments (1)

SteuerzoneAlleLaender.png (11.1 KB ) - added by arno@… 9 years ago.
Screenshot

Download all attachments as: .zip

Change History (9)

by arno@…, 9 years ago

Attachment: SteuerzoneAlleLaender.png added

Screenshot

comment:1 by arno@…, 9 years ago

Nachtrag: Richtig wäre, den ersten "left join" durch einen "inner join" zu ersetzen.

comment:2 by Gerhard Waldemair, 9 years ago

Resolution: fixed
Status: newclosed

In 10230:

fix #996

comment:3 by Torsten Riemer, 9 years ago

Resolution: fixed
Status: closedreopened

Ich öffne das Ticket nochmal, da ich den Fehler im Demoshop 2.0.0.0 rev 9678 nicht nachvollziehen kann.

@Arno: Kannst du mir die Schritte bitte nochmal genau beschreiben, die zu tun sind, um es nachvollziehen zu können?

Last edited 9 years ago by Torsten Riemer (previous) (diff)

comment:4 by arno@…, 9 years ago

Habe gerade versucht, das im Demoshop nachzustellen, ist mir nicht gelungen. Möglicherweise wird aktuell die Steuerzone mit dem Löschen des Landes über das Programm auch gelöscht.

Auf meiner Installation kann ich es nachvollziehen, indem ich das Land in der Datenbank lösche. Damit wird der Datenbestand inkonsistent und der Select greift dann auch die Steuerzonen, für die es kein Land gibt.

Peter Kropff hat den Unterschied sehr schön erklärt:
http://www.peterkropff.de/site/mysql/inner_join.htm
http://www.peterkropff.de/site/mysql/left_right_join.htm

Drauf gekommen bin ich durch einen Kunden, der schon seit längerem selbst bastelt, die Vorgeschichte dieser Datenbank muß grausam sein. ;)

in reply to:  description comment:5 by Torsten Riemer, 9 years ago

Meine Frage war deshalb, da du hier schriebst:

Replying to arno@…:

[...]
Der Fehler tritt sowohl bei 1.06 bis SP4 auf, als auch bei 2.0

Und den Fehler konnte ich da eben schon nicht mehr nachvollziehen.

Kann natürlich sein, dass der Fehler bei dir in einer inkonsistenten Datenbank liegt, wo es noch Zonenzuordnungen zu nicht mehr existierenden Ländern gibt.

Wenn du nun auch schreibst, dass du den Fehler im Demoshop 2.0.0.0 rev 9678 nicht nachvollziehen kannst, dann sollte das doch erledigt sein oder nicht?

Wobei ich mich dann frage, was in r10230 korrigiert wurde. *g*

comment:6 by arno@…, 9 years ago

Ich sage mal ganz klar "jein".

Nein, denn die Korrektur des SQL würde bewirken, dass auch bei inkonsistenter Datenbank die Anzeige richtig wäre.

Ja, denn bei sachgerechter Anwendung kann sich der Fehler nicht auswirken.

Für mich ist der Fall in soweit erledigt, als ich die DB repariert habe.

comment:7 by Torsten Riemer, 9 years ago

Milestone: modified-shop-2.0.1.0modified-shop-2.0.1.1

comment:8 by Gerhard Waldemair, 9 years ago

Resolution: fixed
Status: reopenedclosed

Modify Ticket

Action
as closed The owner will remain somebody.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.