Modify

Opened 10 years ago

Closed 9 years ago

#822 closed Bug/Fehler (fixed)

Kategoriebild mit identischem Namen wird nicht gespeichert

Reported by: Bonsai Owned by: somebody
Priority: hoch Milestone: modified-shop-2.0.2.0
Component: Admin Version: 1.06
Keywords: Cc:
Blocked By: Blocking:

Description

Wenn das Bild identischen Namen hat, ist es schon da , wird gelöscht und dann versucht zu kopieren.

/admin/includes/classes/categories.php

Fix ist eine Abfrage ob Bildname identisch, dann ist der Lösch- und Kopiervorgang obsolet :

      if ($categories_image = xtc_try_upload('categories_image', DIR_FS_CATALOG_IMAGES.'categories/', '777', $accepted_categories_image_files_extensions, $accepted_categories_image_files_mime_types)) {
      $cname_arr = explode('.', $categories_image->filename);
      $cnsuffix = array_pop($cname_arr);
      $categories_image_name = $categories_id.'.'.$cnsuffix;
	  if (DIR_FS_CATALOG_IMAGES.'categories/'.$categories_image->filename !== DIR_FS_CATALOG_IMAGES.'categories/'.$categories_image_name) {
		@ unlink(DIR_FS_CATALOG_IMAGES.'categories/'.$categories_image_name);
		rename(DIR_FS_CATALOG_IMAGES.'categories/'.$categories_image->filename, DIR_FS_CATALOG_IMAGES.'categories/'.$categories_image_name);
	  }

Attachments (0)

Change History (7)

comment:1 by Bonsai, 10 years ago

Der Bug ist eigentlich noch schlimmer!

Wenn man ein 44.png für Kategorie 46.png speichert, ist das Bild von Kategorie 44 weg!

Mein Fix ist deshalb quatsch. Man sollte das Bild in einem anderen Ordner zwischenspeichern oder für das xtc_try_uplaod gleich den richtigen Namen nehmen.

Habe gerade keine Zeit den Fix zu basteln.

comment:2 by Gerhard Waldemair, 10 years ago

Milestone: modified-shop-1.06-SP4modified-shop-2.10
Version: 1.062.10

comment:3 by Torsten Riemer, 10 years ago

Version: 2.0.0.11.06

comment:4 by Torsten Riemer, 9 years ago

Selbes Problem wie in Ticket #993 bzgl. Artikelbildern.

comment:5 by Torsten Riemer, 9 years ago

Milestone: modified-shop-2.1.0.0modified-shop-2.0.1.0
Priority: normalhoch

comment:6 by Torsten Riemer, 9 years ago

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

comment:7 by Gerhard Waldemair, 9 years ago

Resolution: fixed
Status: newclosed

In 10438:

fix #993, #822

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.