linkonoid / oc-shortcodesengine

Core plugin for includes shortcodes in page content
https://octobercms.com/plugin/linkonoid-shortcodesengine
Other
4 stars 3 forks source link

Undefined index: shortcode_values #1

Open sozonovalexey opened 5 years ago

sozonovalexey commented 5 years ago
Undefined index: shortcode_values
/plugins/linkonoid/shortcodesengine/classes/ShortcodesManager.php line 135

Нужно добавить проверку на наличие параметров

linkonoid commented 5 years ago

Да, недочёт вижу – по коду при пустом shortcode_values repeater должна вываливаться ошибка, обязательно поправлю, точнее уже поправил (134 строка):

if (!empty($shortcode['shortcode_values'])) foreach ($shortcode['shortcode_values'] as $key => $value){

исправления выложу чуть позже, спасибо за информацию!

From: Alexey [mailto:notifications@github.com] Sent: Thursday, February 28, 2019 10:20 AM To: linkonoid/octobercms-shortcodesengine Cc: Subscribed Subject: [linkonoid/octobercms-shortcodesengine] Undefined index: shortcode_values (#1)

Undefined index: shortcode_values

/plugins/linkonoid/shortcodesengine/classes/ShortcodesManager.php line 135

Нужно добавить проверку на наличие параметров

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/linkonoid/octobercms-shortcodesengine/issues/1 , or mute the thread https://github.com/notifications/unsubscribe-auth/AVneLxoNpgQdXoQYpWjjTQ9-fED8Jv7iks5vR4MagaJpZM4bWKu2 . https://github.com/notifications/beacon/AVneL-cMDLEfAmYoMNrsH8vPGh5iw5zfks5vR4MagaJpZM4bWKu2.gif


Это сообщение проверено на вирусы антивирусом Avast. https://www.avast.com/antivirus