- Posts: 149
- Thank you received: 6
missing variable safeguard in /components/com_hikashop/router.php
11 years 3 months ago #206270
by icomex
-- HikaShop version -- : 2.5.0
-- Joomla version -- : 3.4.1
-- PHP version -- : 5.3.28
-- Error-message(debug-mod must be tuned on) -- : Notice: Undefined index: task in /home/icxdev04/public_html/components/com_hikashop/router.php on line 53
Since upgrading to Hikashop 2.5.0 I've been seeing ( during checkout )
Notice: Undefined index: task in /home/icxdev04/public_html/components/com_hikashop/router.php on line 53
at the top of the page. Within that router.php file, shouldn't
be
This seems to make sense and makes the issue go away.
-- Joomla version -- : 3.4.1
-- PHP version -- : 5.3.28
-- Error-message(debug-mod must be tuned on) -- : Notice: Undefined index: task in /home/icxdev04/public_html/components/com_hikashop/router.php on line 53
Since upgrading to Hikashop 2.5.0 I've been seeing ( during checkout )
Notice: Undefined index: task in /home/icxdev04/public_html/components/com_hikashop/router.php on line 53
at the top of the page. Within that router.php file, shouldn't
Code:
if( ( isset($query['ctrl']) && $query['ctrl']=='checkout' || isset($query['view']) && $query['view']=='checkout' ) && !empty($query['Itemid']) && ( !isset($query['task']) && !isset($query['layout']) || $query['task']=='step' || (isset($query['layout']) && $query['layout']=='step' )) ) {
be
Code:
if( ( isset($query['ctrl']) && $query['ctrl']=='checkout' || isset($query['view']) && $query['view']=='checkout' ) && !empty($query['Itemid']) && ( !isset($query['task']) && !isset($query['layout']) || (isset($query['task']) && $query['task']=='step' ) || (isset($query['layout']) && $query['layout']=='step' )) ) {
This seems to make sense and makes the issue go away.
Please Log in or Create an account to join the conversation.
11 years 3 months ago #206311
by Xavier
Replied by Xavier on topic missing variable safeguard in /components/com_hikashop/router.php
Hi,
Yes indeed that makes sense, thanks for your report
Yes indeed that makes sense, thanks for your report
The following user(s) said Thank You: icomex
Please Log in or Create an account to join the conversation.
Time to create page: 0.144 seconds