oc-shopaholic / oc-toolbox-plugin Goto Github PK

В методе setPropertiesFields происходит ошибка

ErrorException: array_combine(): Both parameters should have an equal number of elements in /var/www/dm500/plugins/lovata/commercemlshopaholic/classes/import/ImportExchangeOffersFromXML.php:87

Связано это с тем что в в методе prepareImportDataBeforeSave фильтруются значения массива

if (is_string($sValue)) {
   $sValue = trim($sValue);
} elseif (is_array($sValue)) {
   $sValue = array_filter($sValue);
}

Туда после разбора xml при наличии пустого значение в xml приходят такие массивы

array(2) {
  [0]=>
  string(8) "Цвет"
  [1]=>
  string(12) "Размер"
}
string(14) "property_value"
array(2) {
  [0]=>
  string(10) "Серый"
  [1]=>
  string(0) ""
}

Сам xml

<Предложение>
	<Ид>6e70ae10-5967-11eb-a787-00155de01c01#ab82c437-5968-11eb-a787-00155de01c01</Ид>
	<НомерВерсии>AAAAAAAA6JQ=</НомерВерсии>
	<ПометкаУдаления>false</ПометкаУдаления>
	<Наименование>Свитер с горлом (Серый One Size)</Наименование>
	<ХарактеристикиТовара>
		<ХарактеристикаТовара>
			<Наименование>Цвет</Наименование>
			<Значение>Серый</Значение>
		</ХарактеристикаТовара>
		<ХарактеристикаТовара>
			<Наименование>Размер</Наименование>
			<Значение/>
		</ХарактеристикаТовара>
	</ХарактеристикиТовара>
	<ЗначенияСвойств>
		<ЗначенияСвойства>
			<Ид>a34f1e65-58ac-11eb-a786-00155de01c01</Ид>
			<Значение>ab82c436-5968-11eb-a787-00155de01c01</Значение>
		</ЗначенияСвойства>
		<ЗначенияСвойства>
			<Ид>b597ddc0-58ac-11eb-a786-00155de01c01</Ид>
			<Значение/>
		</ЗначенияСвойства>
	</ЗначенияСвойств>
	<Ширина>0</Ширина>
	<Длина>0</Длина>
	<Высота>0</Высота>
</Предложение>

Fix processing of positive results in ComponentSubmitForm::getResponseModeAjax() method, if flash_on enabled

I migrated my project to an S3 storage, and when I try to import models using CSV Import (e.g., shopaholic category) but images are not uploaded.
After some digging, I found out that the problem seems to be in the AbstractImportModelFromCSV helper's initPreviewImage() method on these lines of code:

        $this->sPreviewImage = storage_path($this->sPreviewImage);
        if (!file_exists($this->sPreviewImage)) {
            $this->sPreviewImage = null;
        }

Using storage_path() function it will do a file exist on local storage, when it should check if file exists on S3 storage.
Will there be an update to support uploading images through the AbstractImportModelFromCSV helper to S3 storage, or is there an alternative solution?

Необходимо скопировать с проекта аренды файл ru/lang.php и закоммитить изменения, если они будут.

https://github.com/oc-shopaholic/oc-toolbox-plugin/blob/master/classes/item/MainItem.php#L253

При создании объекта File не создается id , что при некотором стечении обстоятельство может привести к смешиванию картинок превью. Т.к. id используется для уникализации имени.

Как вариант решения

$obFile = File::make($arFileData);
$obFile->disk_name = array_get($arFileData, 'disk_name');
$obFile->id = array_get($arFileData, 'id');

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

Hello,

I'm working with the shopaholic ecosystem. Very good until now except the bad surprise to see that the ajax form validation system completly bypass october way of doing form validation, preventing the use of extra features like inline form validation using <div data-validate-for="my_field"></div> or other very useful features.

In addition, your actual validation system only return the first field that throw an error whereas ValidationException allow to return an array of fields, with an array of messages allowing to do something like that when you submit an empty form with several required fields

To conclude, I think you should provide a new mode when dealing with form component "ajax with validation" that would handle errors the october way. Like that, people that want to use that mode can do it.

Best regards

/**
     * Get element ID list from cache or database
     * @param mixed $sFilterValue
     * @return array|null
     */
    public function get($sFilterValue) : array
    {
        if (empty($sFilterValue)) {
            return [];
        }

        $this->sValue = $sFilterValue;
        if (array_key_exists($this->getCacheKey(), $this->arCachedList) && is_array($this->arCachedList[$this->getCacheKey()])) {
            return $this->arCachedList[$this->getCacheKey()];
        }

        $arElementIDList = $this->getIDList();
        $this->arCachedList[$this->getCacheKey()] = $arElementIDList;

        return $arElementIDList;
    }

Если sFilterValue 0 когда например надо фильтровать по числовым значениям 0,1,2 то фильтрации не происходит

empty($sFilterValue)

будет true

Required improvement for this issue.

Hi, I like to contribute to the localization of this plugin (also other shopaholic plugins).
Would you add Japanese as Crowdin target language?

If you return a custom array via EVENT_BEFORE_IMPORT, this data is not available in functions called by prepareImportData. This is because the event is fired only after prepareImportData.

For example, to ensure that you can edit the preview_image attribute during the import, it is necessary to fire the event before calling the prepareImportData function.

Pull request for this issue:
#91

Any plans to update Toolbox to be compatable with OC3.1, at the moment, many conflicts with Rainlab.Translate plugin, because off many changes, from version 1.x to version 2.x regarding Multisete.

https://github.com/rainlab/translate-plugin/blob/master/UPGRADE.md

I'm running October 3 and I get this error when trying to create my first product with Shopaholic.

Class "RainLab\Translate\Models\Locale" not found" on line 33 of /plugins/lovata/toolbox/traits/helpers/TraitInitActiveLang.php

When I installed Properties for Shopaholic, the error below started appear, and the site cannot be displayed at all.

Class 'Lovata\Toolbox\Classes\Helper\AbstractImportModelFromXML' not found

The cause is that the version of oc-toolbox-plugin installed through composer is a bit old, and thus this class in the error does not exist.
I hope oc-toolbox-plugin on packagist is synched again.

Thanks.