search

Teplitsa / Leyka

Плагин для WordPress, упрощающий сбор, управление и отчетность по онлайн-пожертвованиям, Leyka - Crowdfunding Fundraising and Donations Management System
https://leyka.org
GNU General Public License v2.0
84 stars 51 forks source link

Конкретизировать текст ошибки в эксепшне (Invalid description value) #375

Closed rodion-m closed 2 years ago

rodion-m commented 2 years ago

Обновите, пожалуйста, текст ошибки в случае если указано слишком длинное назначение платежа:

            if ($length > Payment::MAX_LENGTH_DESCRIPTION) {
                throw new InvalidPropertyValueException(
                    'Invalid description value', 0, 'CreatePaymentRequest.description', $value
                );
            }

С 'Invalid description value' на 'Description cannot be longer than ' . Payment::MAX_LENGTH_DESCRIPTION. Так сразу понятно в чем проблема. Вот тут: https://github.com/Teplitsa/Leyka/blob/19649e847e938bb198acf30e37f5b6144c31f479/gateways/yandex/lib/Model/Payment.php#L305

Ahaenor commented 2 years ago

Добрый день! К сожалению, это не наш код - это SDK ЮКассы. Мы не можем вмешиваться в него - даже чтобы сделать что-то лучше.

rodion-m commented 2 years ago

@Ahaenor Да, действительно, это часть SDK ЮКассы. Я проверил и оказалось, что в актуальной версии SDK они конкретизировали текст ошибки: https://git.yoomoney.ru/projects/SDK/repos/yookassa-sdk-php/browse/lib/Model/Payment.php#322 \ Обновите, пожалуйста, SDK ЮКассы в лейке. Можно открыть issue и переименовать ее в "Обновить SDK ЮКассы".