[ticket/102] Fix html validation issues in modules overview

B3P-102
This commit is contained in:
Marc Alexander
2014-04-08 14:35:04 +02:00
parent 83072b0dce
commit e64f04b927
2 changed files with 10 additions and 10 deletions

View File

@@ -671,7 +671,7 @@ class portal_module
'MODULE_IMAGE' => ($row['module_image_src']) ? '<img src="' . $this->root_path . 'styles/' . $this->user->style['style_path'] . '/theme/images/portal/' . $row['module_image_src'] . '" alt="' . $row['module_name'] . '" />' : '',
'MODULE_ENABLED' => ($row['module_status']) ? true : false,
'U_DELETE' => $this->u_action . '&amp;module_id=' . $row['module_id'] . '&amp;module_classname=' . $row['module_classname'] . '&amp;action=delete',
'U_DELETE' => $this->get_module_link('modules', $row['module_id']) . '&amp;action=delete',
'U_EDIT' => $this->get_module_link('config', $row['module_id']),
'U_MOVE_UP' => $this->u_action . '&amp;module_id=' . $row['module_id'] . '&amp;action=move_up',
'U_MOVE_DOWN' => $this->u_action . '&amp;module_id=' . $row['module_id'] . '&amp;action=move_down',
@@ -1199,7 +1199,7 @@ class portal_module
*/
protected function get_module_link($mode, $module_id)
{
return preg_replace(array('/i=[0-9]+/', '/mode=[a-zA-Z0-9_]+/'), array('i=\\' . __CLASS__, 'mode=' . $mode), $this->u_action) . '&amp;module_id=' . $module_id;
return preg_replace(array('/i=[0-9]+/', '/mode=[a-zA-Z0-9_]+/'), array('i=%5C' . str_replace('\\', '%5C', __CLASS__), 'mode=' . $mode), $this->u_action) . '&amp;module_id=' . $module_id;
}
/**

View File

@@ -51,11 +51,11 @@
<table class="portal-modules-overview">
<tr>
<th style="text-align: center;" colspan="3">{L_MODULE_POS_TOP}</th>
<th style="text-align: center;">{L_MODULE_POS_TOP}</th>
</tr>
<!-- BEGIN modules_top -->
<!-- IF not modules_top.MODULE_ENABLED --><tr class="row3"><!-- ELSEIF modules_top.S_ROW_COUNT is even --><tr class="row1"><!-- ELSE --><tr class="row2"><!-- ENDIF -->
<td colspan="3">
<td>
{modules_top.MODULE_IMAGE} {modules_top.MODULE_NAME}<br />
<br />
<span<!-- IF not modules_top.S_FIRST_ROW --> style="display: none;"<!-- ENDIF -->>{ICON_MOVE_UP_DISABLED}&nbsp;</span>
@@ -78,7 +78,7 @@
<table style="background-color: transparent; border: none; padding: 0px; margin: 0px;">
<tr style="vertical-align: top;">
<td style="width: 20%; font-size: 100%; line-height: 100%; padding: 0px; margin: 0px;">
<table cellspacing="1">
<table class="portal-modules-overview">
<tr>
<th>{L_MODULE_POS_LEFT}</th>
</tr>
@@ -107,7 +107,7 @@
</p>
</td>
<td style="font-size: 100%; line-height: 100%; padding: 0 12px;">
<table cellspacing="1">
<table class="portal-modules-overview">
<tr>
<th style="text-align: center;">{L_MODULE_POS_CENTER}</th>
</tr>
@@ -136,7 +136,7 @@
</p>
</td>
<td style="width: 20%; font-size: 100%; line-height: 100%; padding: 0px;">
<table cellspacing="1">
<table class="portal-modules-overview">
<tr>
<th>{L_MODULE_POS_RIGHT}</th>
</tr>
@@ -168,13 +168,13 @@
</table>
<br />
<table cellspacing="1">
<table class="portal-modules-overview">
<tr>
<th style="text-align: center;" colspan="3">{L_MODULE_POS_BOTTOM}</th>
<th style="text-align: center;">{L_MODULE_POS_BOTTOM}</th>
</tr>
<!-- BEGIN modules_bottom -->
<!-- IF not modules_bottom.MODULE_ENABLED --><tr class="row3"><!-- ELSEIF modules_bottom.S_ROW_COUNT is even --><tr class="row1"><!-- ELSE --><tr class="row2"><!-- ENDIF -->
<td colspan="3">
<td>
{modules_bottom.MODULE_IMAGE} {modules_bottom.MODULE_NAME}<br />
<br />
<span<!-- IF not modules_bottom.S_FIRST_ROW --> style="display: none;"<!-- ENDIF -->>{ICON_MOVE_UP_DISABLED}&nbsp;</span>