Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#2218 closed Bug/Fehler (fixed)

Fehler im Datenbankquery für "Neue Artikel"

Reported by: Karl1 Owned by: somebody
Priority: normal Milestone: modified-shop-2.0.7.1
Component: Shop Version: 2.0.7.0
Keywords: Cc:
Blocked By: Blocking:

Description

Problem besteht bei Aufruf des Links "Neue Artikel".

In Version 2.0.6.0 hatte ich noch 6 neue Artikel, in 2.0.7.0 habe ich 38 Artikel - mein Testshop hat nur 6 Artikel alles andere sind Verlinkungen.
Habe daraufhin die Datenbankqueries der Dateien includes/modules/default.php verglichen - in 2.0.7.0 fehlt "GROUP BY p.products_id".

Gruß Karl

P.S.: Screenshot angehängt.

Attachments (3)

screen.png (115.6 KB ) - added by Karl1 4 years ago.
Screenshot
changeset_14482.zip (3.6 KB ) - added by Gerhard Waldemair 4 years ago.
changeset_14482
changeset_14525.zip (3.7 KB ) - added by Torsten Riemer 4 years ago.

Download all attachments as: .zip

Change History (9)

by Karl1, 4 years ago

Attachment: screen.png added

Screenshot

comment:1 by Torsten Riemer, 4 years ago

Milestone: modified-shop-2.0.7.1

comment:2 by anonymous, 4 years ago

Es geht um die Zeile

".((isset($subcategories_array) && count($subcategories_array) > 1) ? 'GROUP BY p.products_id' : '')."

die dann wahrscheinlich so lauten müsste

".(((isset($subcategories_array) && count($subcategories_array) > 1) || basename($PHP_SELF) == FILENAME_PRODUCTS_NEW) ? 'GROUP BY p.products_id' : '')."

Es gibt allerdings noch eine andere Ungereimtheit im Zusammenhang mit der products_new.php, die es allerdings auch bereits in der 2.0.6.0 gab.

Dazu gibt es ein neues Ticket, welches ich hier verlinke weil die Themen als zusammengehörig angesehen werden können: Ticket #2221

Gruß,
noRiddle

Version 0, edited 4 years ago by anonymous (next)

comment:3 by Karl1, 4 years ago

Selbes Problem trifft auf "Angebote" zu.

'GROUP BY p.products_id' behebt den Fehler.

Gruß Karl

comment:4 by noRiddle, 4 years ago

Vielleicht sollte man neue Variablen einführen:

$pages_with_group_by = array(FILENAME_PRODUCTS_NEW, FILENAME_SPECIALS);
$use_group_by = (isset($subcategories_array) && count($subcategories_array) > 1) || (in_array(basename($PHP_SELF), $pages_with_group_by));

und dann

".((isset($use_group_by) && $use_group_by === true) ? 'GROUP BY p.products_id' : '')."

Gruß,
noRiddle

comment:5 by Gerhard Waldemair, 4 years ago

Resolution: fixed
Status: newclosed

In 14482:

fix #2218 fix wrong pagination

by Gerhard Waldemair, 4 years ago

Attachment: changeset_14482.zip added

changeset_14482

comment:6 by Gerhard Waldemair, 4 years ago

In 14525:

fix #2218 fix wrong pagination

by Torsten Riemer, 4 years ago

Attachment: changeset_14525.zip added

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.