Seite 1 von 1

[CON-395] Gelöst: Überflüssiger Code include.lay_overview...

Verfasst: Di 1. Mär 2011, 13:59
von rethus
In besagter Datei ist direkt am Anfang folgender Code zu finden:

Code: Alles auswählen

while ($layout = $layouts->next()) {

    if($perm->have_perm_area_action_item("lay_edit","lay_edit",$layout->get("idlay"))){      //idlay of area lay is 8

        $name  = $layout->get('name');
        $descr = $layout->get('description');
        $idlay = $layout->get('idlay');

        if (strlen($descr)  > 64) {
            $descr = substr($descr, 0, 64);
            $descr .= ' ..';
        }

        if ($perm->have_perm_area_action_item("lay_edit","lay_edit",$layout->get("idlay"))) {
        	
        	$tmp_mstr = '<a href="javascript:conMultiLink(\'%s\', \'%s\', \'%s\', \'%s\')" title="%s" alt="%s">%s</a>';
        	$area = "lay";
        	$mstr = sprintf($tmp_mstr, 'right_top',
                                       $sess->url("main.php?area=$area&frame=3&idlay=$idlay"),
                                       'right_bottom',
                                       $sess->url("main.php?area=lay_edit&frame=4&idlay=$idlay"),
                                       $descr, $descr, $name);

            $tpl->set('d', 'NAME',  $mstr);
        } else {
        	//$tpl->set('d', 'IMGLINK', '<img src="images/layout.gif" width="16" height="16">');
            $tpl->set('d', 'NAME',  $name);
        }
Diese IF-Abfrage ist in sich selbst geschachtelt:
Nach dem While, wird diese IF-Schleife betreten...

Code: Alles auswählen

 if($perm->have_perm_area_action_item("lay_edit","lay_edit",$layout->get("idlay"))){
und innerhalb dieser IF-Schleife finden wir erneut:

Code: Alles auswählen

 if($perm->have_perm_area_action_item("lay_edit","lay_edit",$layout->get("idlay")))
Ein else-Befehl, welcher niemals ausgeführt wird....
Bitte in den Bugtracker und Code bereinigen.

Re: Überflüssiger Code include.lay_overview.inc.php

Verfasst: Di 1. Mär 2011, 14:22
von Oldperl
:!: Zur Prüfung verschoben nach Bugs

Gruß aus Franken

Ortwin

Re: Überflüssiger Code include.lay_overview.inc.php

Verfasst: Fr 18. Mär 2011, 23:40
von xmurrix
Ist gefixt.