Jump to content



Issue Information

  • #000336

  • 1 - Low

  • Fixed

  • 2.3.1

  • 2.3.3

Issue Confirmations

  • Yes (0)No (0)
Photo

page not set in action_recorder php

Posted by Gergely on 17 August 2011 - 14:15

Hi Mark,


Broken links after second pages and cant navigate in action_recorder.php. The 'page' is not set.


echo '              	<tr class="dataTableRow" onmouseover="rowOverEffect(this)" onmouseout="rowOutEffect(this)" onclick="document.location.href=\'' . tep_href_link(FILENAME_ACTION_RECORDER, 'aID=' . $actions['id'] . (isset($HTTP_GET_VARS['page']) ? '&page=' . $HTTP_GET_VARS['page'] : '')) . '\'">' . "\n";


<td class="dataTableContent" align="right"><?php if ( (isset($aInfo) && is_object($aInfo)) && ($actions['id'] == $aInfo->id) ) { echo tep_image(DIR_WS_IMAGES . 'icon_arrow_right.gif', ''); } else { echo '<a href="' . tep_href_link(FILENAME_ACTION_RECORDER, 'aID=' . $actions['id'] . (isset($HTTP_GET_VARS['page']) ? '&page=' . $HTTP_GET_VARS['page'] : '')) . '">' . tep_image(DIR_WS_IMAGES . 'icon_info.gif', IMAGE_ICON_INFO) . '</a>'; } ?>&nbsp;</td>

Regadrs
Gergely

Hi again,

and not stabilized modules and filters too


Regards,
Gergely

Updating severity to: 2 - Fair
Updating status to: Confirmed

Updating status to: Fixed
Issue fixed in: 2.3.2

Fixed in https://github.com/o...ff6ae1b0c8275e7

Thanks for the report

Updating status to: Fixed
Issue fixed in: 2.3.2

Fixed in https://github.com/o...ff6ae1b0c8275e7

Thanks for the report


Thanks Mark!

This is not enough on icon_info.gif need the same change.

<td class="dataTableContent" align="right"><?php if ( (isset($aInfo) && is_object($aInfo)) && ($actions['id'] == $aInfo->id) ) { echo tep_image(DIR_WS_IMAGES . 'icon_arrow_right.gif', ''); } else { echo '<a href="' . tep_href_link(FILENAME_ACTION_RECORDER, 'aID=' . $actions['id']) . '">' . tep_image(DIR_WS_IMAGES . 'icon_info.gif', IMAGE_ICON_INFO) . '</a>'; } ?>&nbsp;</td>

Regards,
Gergely

Thanks, I have just committed a fix for the info icon as well.

changed fixed-in version to: 2.3.3

changed severity to: 1 - Low