#2277 closed Bug/Fehler (fixed)
VPE Name wird nicht angezeigt ab Version 2.0.7.0
| Reported by: | anonymous | Owned by: | somebody |
|---|---|---|---|
| Priority: | normal | Milestone: | modified-shop-2.0.8.0 |
| Component: | Admin | Version: | 2.0.7.0 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
In der /inc/xtc_get_vpe_name.inc.php gibt es einen Fehler, sodass die VPE Name nicht angezeigt wird. Der Fehler betrifft nur Versionen 2.0.7.0, 2.0.7.1 und 2.0.7.2. Davor war die Funktion nur eine Datenbankabfrage.
Den Fehler habe ich im aktuellen Demoshop nachstellen können.
Erst xtc_db_num_rows mit dem zweiten Parameter true bringt das gewünschte Ergebnis.
Desweiteren eine Bedingung, welche nie ausgeführt wird
Statt isset($vpe_name_array)... ist die Prüfung auf is_array($vpe_name_array)... sinnvoller, da $vpe_name_array bereits static definiert.
Screenshot und die geänderte Funktion habe ich angehangen
Attachments (7)
Change History (12)
by , 3 years ago
| Attachment: | Screenshot 2022-07-13 215541.png added |
|---|
by , 3 years ago
| Attachment: | xtc_get_vpe_name.inc.php added |
|---|
comment:1 by , 3 years ago
by , 3 years ago
| Attachment: | 827179CB-F06C-4C34-A713-A488105E58C9.png added |
|---|
comment:2 by , 3 years ago
Natürlich habe ich Stück eingestellt und den Artikel in einem privaten Fenster aufgerufen. Shopcaches auch gelöscht.
by , 3 years ago
| Attachment: | Screenshot 2022-07-14 081034.png added |
|---|
by , 3 years ago
| Attachment: | Screenshot 2022-07-14 081441.png added |
|---|
by , 3 years ago
| Attachment: | Screenshot 2022-07-14 080952.png added |
|---|
comment:3 by , 3 years ago
Bei so viel Beweis...
du hast recht, da fehlt der Parameter. Das wird gleich korrigiert.
Eine Variable die als STATIC deklariert ist, ist nicht automatisch auch definiert. Die Variable kann ja alles mögliche sein. Array, String, Object, etc... Deshalb ist das mit isset korrekt und wird auch einmalig ausgeführt.
comment:5 by , 3 years ago
| Version: | 2.0.7.2 → 2.0.7.0 |
|---|
by , 3 years ago
| Attachment: | changeset_14658.zip added |
|---|
Das scheint nur zu passieren, wenn man bei VPE keinen Wert wie Stück angibt, sondern das Feld auf „keine“ belässt. Gibt man eine VPE an, wird diese in Demoshop auch angezeigt.