Разработчикам

Удаление значений множественного свойства типа ФАЙЛ

RSS

Пользователь 8489

Посетитель

Сообщений: 44 Баллов: 7 Регистрация: 13.01.2007

07.11.2007 01:54:02

Проблема в следующем:
у меня есть множественное свойство типа файл - MORE_PHOTO.
В другое множественное свойство PHOTO_SM я копирую картинки из свойства MORE_PHOTO поэлементно, при этом уменьшая их.
Всё формируется, всё работает, но когда я сохраняю изменения через CIBlockElement::SetPropertyValues - набор свойств постоянно множится, удваиваясь.
Я пришел к выводу, что необходимо ПОЛНОСТЬЮ чистить это свойство перед тем, как что-либо туда записывать.
Но вот загвоздка - как это сделать?
Прочитал, что для файлового массива необходимо добавить ["del"]="Y", тогда, вроде как, файл будет удален.
Вот кусок кода. Может быть, кто-нибудь подскажет, как, что и где исправить?

Код

$db_props = CIBlockElement::GetProperty($arFields["IBLOCK_ID"], $arFields['ID'], "sort", "asc", Array("CODE"=>"PHOTO_SM"));

   while($ar_props = $db_props->Fetch())
   {
   
   if ($ar_props["VALUE"])
      {
     $ar_val = $ar_props["VALUE"];
      $ar_val_id = $ar_props["PROPERTY_VALUE_ID"];
      // формируем путь к файлу картинки путем сложения адреса сайта и внутреннего пути к картинке
      $img_path2 = $_SERVER['DOCUMENT_ROOT'].CFile::GetPath($ar_props["VALUE"]);
   $new_make_file = CFile::MakeFileArray($img_path2);
     $new_make_file["del"]="Y";
     CIBlockElement::SetPropertyValueCode($arFields["ID"], "PHOTO", Array ($ar_val_id => $new_make_file) );
   

      }      
   }

Пользователь 8489

Посетитель

Сообщений: 44 Баллов: 7 Регистрация: 13.01.2007

07.11.2007 14:51:38

Привожу весь текст скрипта.
Назначение его такого. У нас имеется множественное свойство MORE_PHOTO. Скрипт должен сгенерировать привьюшки для всех изображений этого свойства, а затем эти привьюшки записать в свойство PHOTO_SM.
Скрипт удалось заставить заработать, но не без маленького глюка. Первый раз отрабатывает прекрасно. Но вот при повторном сохранении элемента инфоблока в полях PHOTO_SM вылазят пустые поля, причем, количество их в точности совпадает с количеством элементов в MORE_PHOTO.

Кто сможет помочь это исправить - буду благодарен.

Код

<?
function BXIBlockAfterSave($arFields)
{
// Вначале ПОЛНОСТЬЮ очищаем свойство PHOTO_SM
$db_props = CIBlockElement::GetProperty($arFields["IBLOCK_ID"], $arFields['ID'], "sort", "asc", Array("CODE"=>"PHOTO_SM"));
//print_r ($db_props);die();
   while($ar_props = $db_props->Fetch())
   {

   if ($ar_props["VALUE"])
      {
     $ar_val = $ar_props["VALUE"];
      $ar_val_id = $ar_props["PROPERTY_VALUE_ID"];

      // формируем путь к файлу картинки путем сложения адреса сайта и внутреннего пути к картинке
      $img_path2 = $_SERVER['DOCUMENT_ROOT'].CFile::GetPath($ar_props["VALUE"]);
     $new_make_file = CFile::MakeFileArray($img_path2);
     $new_make_file["del"]="Y";
      CIBlockElement::SetPropertyValueCode($arFields["ID"], "PHOTO_SM", Array ($ar_val_id => Array("VALUE"=>$new_make_file) ) );
      @unlink($new_make_file);

      }      
   }
   


// Стандартный пример - создание изображения для PRIVIEW_PICTURE из DETAIL_PICTURE
if(1==1 || $_POST['MAKE_PREVIEW_PICTURE']=='Y')
    {
        $dbr = CIBlockElement::GetByID($arFields['ID']);
        if(($ar = $dbr->Fetch()) && $ar['DETAIL_PICTURE']>0)
        {
            $img_path = $_SERVER['DOCUMENT_ROOT'].CFile::GetPath($ar['DETAIL_PICTURE']);
            $width = 200;
            $height = 200;
            list($width_orig, $height_orig) = getimagesize($img_path);
            if($width && ($width_orig < $height_orig))
               $width = ($height / $height_orig) * $width_orig;
            else
               $height = ($width / $width_orig) * $height_orig;
            $image_p = imagecreatetruecolor($width, $height);
            $image = imagecreatefromjpeg($img_path);
            imagecopyresized($image_p, $image, 0, 0, 0, 0, 
                             $width, $height, $width_orig, $height_orig);
$tmpnm = mt_rand();
            $new_img_path = tempnam("/tmp", "FOO").$tmpnm.".jpg";

            imagejpeg($image_p, $new_img_path);
            $be = new CIBlockElement();
            $be->Update($arFields['ID'], 
                        Array('PREVIEW_PICTURE' => CFile::MakeFileArray($new_img_path)), 
                        false);
            @unlink($new_img_path);
        }
    }


// Самое интересное - берем элементы из MORE_PHOTO, ресайзим и добавляем в PHOTO_SM
$db_props = CIBlockElement::GetProperty($arFields["IBLOCK_ID"], $arFields['ID'], "sort", "asc", Array("CODE"=>"MORE_PHOTO"));

$arFils=array();
while($ar_prop1 = $db_props->Fetch())
{
   $ar_prop=$ar_prop1["VALUE"];

    $k=CFile::GetPath($ar_prop);


    $img_path2 = $_SERVER['DOCUMENT_ROOT'].$k;
   $width  = 200;
      $height = 150;
      list($width_orig, $height_orig) = getimagesize($img_path2);

      if ($width_orig > $width || $height_orig > $height)
         {
         if($width && ($width_orig < $height_orig))
            $width = ($height / $height_orig) * $width_orig;
         else
            $height = ($width / $width_orig) * $height_orig;
         $image_p = imagecreatetruecolor($width, $height);
         $image = imagecreatefromjpeg($img_path2);
         imagecopyresampled($image_p, $image, 0, 0, 0, 0, $width, $height, $width_orig, $height_orig);
$tmpnm = mt_rand();
       $new_img_path = tempnam("/tmp", "FOO").$tmpnm.".jpg";;
         imagejpeg($image_p, $new_img_path);
       $new_make_file = CFile::MakeFileArray($new_img_path);
       echo "New path:".$new_img_path."<br>";
                 
       $arFils=array(
          'VALUE' => $new_make_file);
       CIBlockElement::SetPropertyValues( $arFields['ID'], $arFields["IBLOCK_ID"], $arFils, "PHOTO_SM");
            @unlink($new_img_path);
      }

}
}?>

Пользователь 14162

Посетитель

Сообщений: 83 Баллов: 9 Регистрация: 26.07.2007

08.11.2007 08:52:12

Чистить перед записью ничего не надо. Чтобы не удваивалось, нужно использовать ID записи свойства (PROPERTY_VALUE_ID) в качестве ключа масива для изменения.
Для удаеления ключу VALUE нужно присвоить Array("del" => "Y").
Пример: $arr[$ar_props['PROPERTY_VALUE_ID']] = Array("VALUE" => Array("del" => "Y"));
Таким образом будет удалена запись в множественном свойстве.
Файл нужно удалять отдельно.
CFile::Delete($ar_props['VALUE']),
где props['VALUE'] - идентификатор зарегистрированного в Битрикс файла, это то, что хранится в записи множественного свойства.

Пользователь 8489 Посетитель Сообщений: 44 Баллов: 7 Регистрация: 13.01.2007	#4 12.11.2007 12:07:39 Спасибо огромное! всё заработало прекрасно!

Пользователь 12208 Постоянный посетитель Сообщений: 198 Баллов: 19 Регистрация: 25.05.2007	#5 26.11.2007 01:42:06 nietzschean Расскажите как в вашем примере вы применили совет nikki21 У меня точно такая же задача как у nietzschean.

Пользователь 8489

Посетитель

Сообщений: 44 Баллов: 7 Регистрация: 13.01.2007

28.11.2007 12:33:43

Вот полный код написанного скрипта, глюков, подобных описанным, в нём не обнаружил.

В начале я чищу все значения множественного свойства, сразу замечу - я сделал на форме "флажок", и только когда он включен - происходит данный механизм

Код

<?
function BXIBlockAfterSave($arFields)
{

if($_POST['MAKE_SM_PREVIEW_PICTURES']=='Y')
{
    $db_props = CIBlockElement::GetProperty($arFields["IBLOCK_ID"], $arFields['ID'], "sort", "asc", Array("CODE"=>"PHOTO_SM"));
   while($ar_props = $db_props->Fetch())
   {

   if ($ar_props["VALUE"])
      {
      $ar_val = $ar_props["VALUE"];
      $ar_val_id = $ar_props["PROPERTY_VALUE_ID"];

      $arr[$ar_props['PROPERTY_VALUE_ID']] = Array("VALUE" => Array("del" => "Y"));
      CIBlockElement::SetPropertyValueCode($arFields["ID"], "PHOTO_SM", $arr );
      CFile::Delete($ar_props['VALUE']);
      @unlink($new_make_file);

      }      
   }

То есть, в соответствии с советом, я создаю массив $arr и в нем прописываю поле "del", затем через SetProperyValueCode сохраняю изменения, далее - удаляю файл:

Код
$arr[$ar_props['PROPERTY_VALUE_ID']] = Array("VALUE" => Array("del" => "Y")); CIBlockElement::SetPropertyValueCode($arFields["ID"], "PHOTO_SM", $arr ); CFile::Delete($ar_props['VALUE']);

На втором этапе - записываю все файлы заново

Код

$db_props = CIBlockElement::GetProperty($arFields["IBLOCK_ID"], $arFields['ID'], "sort", "asc", Array("CODE"=>"MORE_PHOTO"));

$arFils=array();
while($ar_prop1 = $db_props->Fetch())
{
    $ar_prop=$ar_prop1["VALUE"];

    $k=CFile::GetPath($ar_prop);


    $img_path2 = $_SERVER['DOCUMENT_ROOT'].$k;
    $width  = 200;
      $height = 150;
      list($width_orig, $height_orig) = getimagesize($img_path2);

      if ($width_orig > $width || $height_orig > $height)
         {
         if($width && ($width_orig < $height_orig))
            $width = ($height / $height_orig) * $width_orig;
         else
            $height = ($width / $width_orig) * $height_orig;
         $image_p = imagecreatetruecolor($width, $height);
         $image = imagecreatefromjpeg($img_path2);
         imagecopyresampled($image_p, $image, 0, 0, 0, 0, $width, $height, $width_orig, $height_orig);
$tmpnm = mt_rand();
         $new_img_path = tempnam("/tmp", "FOO").$tmpnm.".jpg";;
         imagejpeg($image_p, $new_img_path);
         $new_make_file = CFile::MakeFileArray($new_img_path);
                 
         $arFils=array(
            'VALUE' => $new_make_file);
         CIBlockElement::SetPropertyValues( $arFields['ID'], $arFields["IBLOCK_ID"], $arFils, "PHOTO_SM");
            @unlink($new_img_path);
        }
}
}
}?>

Пользователь 165965

Заглянувший

Сообщений: 9 Баллов: 1 Регистрация: 14.02.2013

14.10.2013 14:18:10

Приведу еще раз код, который удаляет один файл из множества в свойстве ИБ:
PS: мой вариант особо не отличается от того, что писали выше, просто лучше оформлен для понимания.

FILES - Имя свойства в ИБ
$nIBlockID - ID ИБ
$nElementID- ID элемента ИБ

Код

/*
 * Получаем свойства "пользовательского свойства" по его коду
 */
$rsProperty = $el->GetProperty($nIBlockID, $nElementID, false, false, array('CODE' => 'FILES'));

// Проходимся по каждой записи множественного свойства
while($arProperty = $rsProperty->Fetch()) {

    // Если переданный ID файла на удаление совпадает со значением свойства, то удаляем его
    if($arProperty['VALUE'] == (int)$_REQUEST['FID']) {

        // Удаляем файл (обновляем запись в ИБ)
        $el->SetPropertyValues(
            $nElementID,
            $nIBlockID,
            array(
                $arProperty['PROPERTY_VALUE_ID'] => array(
                    "VALUE" => array("del" => "Y")
                )
            ),
            'FILES' 
        );

        break;
    }
}

Пользователь 165965

Заглянувший

Сообщений: 9 Баллов: 1 Регистрация: 14.02.2013

14.10.2013 14:29:36

PS: Для удаления всех записей множественного свойства файлов:

Код
$el->SetPropertyValuesEx( $nElementID, $nIBlockID, array('FILES' => array('VALUE' => array())) );

Пользователь 178181

Постоянный посетитель

Сообщений: 115 Баллов: 15 Регистрация: 03.04.2013

14.10.2013 17:04:03

Уважаемый nietzschean, а чем вас не устроили функции для работы с изображениями CFile::ResizeImage и CFile::ResizeImageGet в условиях поставленной задачи? Или это для примера было вообще?
Свойств это не касается, но интересен ваш опыт, так как сам часто с изображениями работаю.

Мальчик был настолько ленив, что специально вставал пораньше - чтобы подольше ничего не делать! (с)

Пользователь 288376

Заглянувший

Сообщений: 12 Регистрация: 09.11.2014

#10

20.01.2015 12:54:13

А обязательно удалять файл

Код
CFile::Delete($file_id);

Или

Код
$el->SetPropertyValuesEx( $nElementID, $nIBlockID, array('FILES' => array('VALUE' => array("del" => "Y"))) );

тоже удаляет сам файл?

Пользователь 229369

Заглянувший

Сообщений: 2 Регистрация: 02.12.2013

#11

18.02.2015 15:37:35

Цитата

Юрій Солтис написал:
А обязательно удалять файл

Код
CFile::Delete($file_id);

Или

Код
$el->SetPropertyValuesEx( $nElementID, $nIBlockID, array('FILES' => array('VALUE' => array("del" => "Y"))) );

тоже удаляет сам файл?

Нет при использовании array('VALUE' => array("del" => "Y" ) файл сам удаляется с диска. (проверял лично)

А если удалить файл через:

Код
CFile::Delete($file_id);

то в инфоблоке останется строчка файл не найден и на сайте будет выводиться битая картинка.

Пользователь 17090

Посетитель

Сообщений: 53 Баллов: 4 Регистрация: 29.10.2007

BRAND & BUSINESS

#12

29.03.2017 22:55:59

Еще пример кода удаления картинок и заливки новых:

Код

// $arIBFields получено путем обычного гетлиста, свойство с кодом IMG содержит много картинок, которые нужно удалить и заменить на новые,
// которые хранятся в массиве $arImages (массив подготовлен при помощи инструкции https://dev.1c-bitrix.ru/learning/course/index.php?COURSE_ID=43&LESSON_ID=5534)

$arDeleteOldImages = array();
foreach($arIBFields['PROPERTY_IMG_PROPERTY_VALUE_ID'] as $old_file) {
   $arDeleteOldImages[$old_file] = Array("VALUE" => Array("del" => "Y"));
}
CIBlockElement::SetPropertyValueCode($arFields['ID'], 'IMG', $arDeleteOldImages);
CIBlockElement::SetPropertyValueCode($arFields['ID'], 'IMG', $arImages);

Пользователь 774815

Посетитель

Сообщений: 33 Баллов: 3 Регистрация: 29.11.2016

#13

20.04.2017 03:17:16

Цитата
Нет при использовании array('VALUE' => array("del" => "Y" ) файл сам удаляется с диска. (проверял лично)

Подтверждаю, удаляется все связанное с файлом, включая сам файл.

Пользователь 365107

Посетитель

Сообщений: 47 Баллов: 4 Регистрация: 29.07.2015

#14

26.03.2018 16:29:06

У меня ни один код из ветки не завёлся, потому что функция АВТОР не очищала массив, а дополненный постоянно сохраняла. Битрикс не готов к такому беспределу, поэтому я написал на один запрос на удаление.

Код

function removeMoreImage($elementId = 0, $iblockId = 0) {
   // Получаем данные из БД
   $dbProps = \CIBlockElement::GetProperty($iblockId, $elementId, 'sort', 'asc', ['CODE' => 'MORE_IMAGE']);
   $arr = [];
   // Разбираем построчно данные из БД
   while ($arItem = $dbProps->Fetch()) {
      // Если указан ключ VALUE, то есть картинка
      if ($arItem['VALUE']) {
         // Так, набираем массив, для удаления
         $arr[$arItem['PROPERTY_VALUE_ID']] = ['VALUE' => ['del' => 'Y']];
         // Не похоже, что без этого фотографии действительно удаляются, лучше проверить
         $res = CFile::Delete($arItem['VALUE']);
      }
   }

   // Одной вилкой чистим множественной свойство картинок
   \CIBlockElement::SetPropertyValueCode(
      $elementId,
      'MORE_IMAGE',
      $arr
   );
}

Пользователь 1762335

Заглянувший

Сообщений: 3 Баллов: 1 Регистрация: 12.02.2018

#15

31.07.2019 09:21:20

Как называется, когда ты перерыл всю документацию, потом два дня гуглил, и только потом нашел единственный пост, где объясняется, как выполнить базовую операцию удаления?
Уровень официальной документации никакой.

Пользователь 3805204

Заглянувший

Сообщений: 1 Регистрация: 02.03.2020

#16

18.05.2020 21:05:34

На сайте вставили новые фото и удалили несколько старых, обновление множественного поля типа файл прошло так.

$el = new CIBlockElement; // получим новый массив с сайта
$arFileUpdate = $params->get('ARR_PHOTOS_FROM_SILE');
// получим массив старых файлов
$resProp = $el->GetProperty(IBLOCK_ID, $v_el, Array("sort" => "asc"), Array("ACTIVE" => "Y", "CODE" => "PHOTOS"));
if ($resProp) {
while ($prop = $resProp->Fetch()) {
if (!in_array($prop['VALUE'], $arFileUpdate)) {
// Удаляем файл которого нет в новом массиве
$el->SetPropertyValues(
$v_el,
IBLOCK_ID,
array(
$prop['PROPERTY_VALUE_ID'] => array(
"VALUE" => array("del" => "Y")
)
),
'PHOTOS'
);
} else {
// Удаляем файл из массива если он уже есть в старом масссиве

unset($arFileUpdate[array_search($prop['VALUE'], $arFileUpdate)]);
}
}
}
// то что осталось записываем
$arFields = array();
$arFields['NAME'] = $params->get('NAME'.$k_el);
$arFields['PROPERTY_VALUES'] = [
'SB_USER_DOC_SCAN' => $arFileUpdate,
];
$res = $el->Update($v_el, $arFields);

здесь не надо чистить весь массив старых файлов и заново записывать новыми значениями

Пользователь 1107293 Заглянувший Сообщений: 2 Регистрация: 25.04.2017	#17 28.01.2022 00:20:29 Вот здесь описано короткое решение. Задача 4: удаление определенного значения множественного свойства типа файл: https://dev.1c-bitrix.ru/learning/course/index.php?COURSE_ID=43&LESSON_ID=5534#%22value_del

Пользователь 6211640

Заглянувший

Сообщений: 7 Регистрация: 27.04.2022

#18

01.11.2023 20:31:24

Цитата
написал: Чистить перед записью ничего не надо. Чтобы не удваивалось, нужно использовать ID записи свойства (PROPERTY_VALUE_ID) в качестве ключа масива для изменения.

Здравствуйте, подскажите, пожалуйста, как перезаписать значения множественного свойства типа файл чтобы не удваивалось и при этом не удалять перед записью? Можно прям пример кода, никак не могу сообразить. В документации написано что надо использовать метод SetPropertyValuesEx вместо SetPropertyValueCode ("Отличие метода SetPropertyValueCode от SetPropertyValuesEx в том, что он не перезаписывает множественное свойства типа Файл, а дополняет."), но с ним (SetPropertyValuesEx) тоже ничего не получается.

Удаление значений множественного свойства типа ФАЙЛ

Продукты

Управление сайтом

Битрикс24

Интернет-магазин + CRM

Решения

Для интернет-магазинов

Каталог готовых решений

Внедрение

Выбрать партнера

Проверить партнера

Стать партнером