Unterkategorie verschoben, Kategoriebilder weg, images/categories nun 644

[pidot]

Hallo Zusammen,

System V 2.0.1.0
Kunde meldete: Kategoriebilder weg.
Was hat er getan ? Er hat eine Unterkategorie in eine andere Oberkategorie verschoben und die Bilder aller Kategorien wurden nicht mehr angezeigt. Nach ein wenig Suche fand ich heraus, dass der Ordner images/categories nun auf einmal 644 bei den Berechtigungen besaß. Alle enthaltenen Bilder ebenso.
Es scheint da irgendwo beim verschieben der Kategorie ein CHMOD 644 gelaufen zu sein.
Ich habe es selber noch nicht testen können, aber bei besagtem Kunden ist es nun schon zum zweiten Mal aufgetreten...

Grüße
Peter

Linkback: https://www.modified-shop.org/forum/index.php?topic=36353.0

[hpzeller]

Hallo Peter,

es scheint die Ursache ist ein Fehler im Code, und zwar in der '/admin/includes/classes/categories.php' ab Zeile 319 steht folgende Codesequenz,

Code: PHP  [Auswählen]

      //copy+rename image
      $src_pic = DIR_FS_CATALOG_IMAGES.'categories/'.$ccopy_values['categories_image'];
      if (is_file($src_pic)) {
        $get_suffix = explode('.', $ccopy_values['categories_image']);
        $suffix = array_pop($get_suffix);
        $categories_image_name = $categories_image_name_process = $this->image_name($new_cat_id, '', $cnsuffix, $cname_arr, false, $sql_data_array);
        @copy($src_pic, DIR_FS_CATALOG_IMAGES.'categories/'.$dest_pic);
        @chmod(DIR_FS_CATALOG_IMAGES.'categories/'.$dest_pic, 0644);

        $this->catModules->copy_category_image($src_pic,$dest_pic);

        //write to DB
        xtc_db_query("UPDATE ".TABLE_CATEGORIES."
                         SET categories_image = '".xtc_db_input($dest_pic)."'
                       WHERE categories_id = '".$new_cat_id."'");
      }
 

die in diesem Code verwendete Variable '$dest_pic' wird nicht definiert, deshalb gibt es einen CHMOD 644 für das Verzeichnis '/images/categories', des Weiteren werden natürlich die zusätzlichen Bilder beim Kopieren durch duplizieren auch nicht erzeugt und auch nicht in die Datenbank eingetragen.

Tipp:
Ersetze mal obigen Code mit diesem

Code: PHP  [Auswählen]

      //copy+rename image
      $src_pic = DIR_FS_CATALOG_IMAGES.'categories/'.$ccopy_values['categories_image'];
      if (is_file($src_pic)) {
        $get_suffix = explode('.', $ccopy_values['categories_image']);
        $suffix = array_pop($get_suffix);
        $dest_pic = $this->image_name($new_cat_id, '', $suffix, $get_suffix, $src_category_id);
        @copy($src_pic, DIR_FS_CATALOG_IMAGES.'categories/'.$dest_pic);
        @chmod(DIR_FS_CATALOG_IMAGES.'categories/'.$dest_pic, 0644);

        $this->catModules->copy_category_image($src_pic,$dest_pic);

        //write to DB
        xtc_db_query("UPDATE ".TABLE_CATEGORIES."
                         SET categories_image = '".xtc_db_input($dest_pic)."'
                       WHERE categories_id = '".$new_cat_id."'");
      }
 

Gruss
Hanspeter

[hpzeller]

Nachtrag:

Das beschriebene Problem  kann ich reproduzieren, allerdings nicht beim Verschieben sondern beim Kopieren von Kategorien.

Gruss
Hanspeter

[web28]

Ist tatsächlich ein Bug und wurde behoben wie im Ticket #1114 beschrieben.

Gruss Web28