Seite 2 von 3

Verfasst: Mo 15. Dez 2003, 11:30
von timo
emergence hat geschrieben:in contenido\includes\grouprights.inc.php
findet man

Code: Alles auswählen

WHERE A.idarea!='login'
sollte aber

Code: Alles auswählen

WHERE A.name!='login'
sein.
Ist gefixt

Verfasst: Mo 15. Dez 2003, 11:32
von timo
emergence hat geschrieben:tja offen sind noch:
in includes.con_art_overview.php

Code: Alles auswählen

$perm->have_perm_area_action(1)) && $perm->have_perm_item(6, $idcat)) ||

ersetzen durch

Code: Alles auswählen

$perm->have_perm_area_action("con")) && $perm->have_perm_item("str", $idcat)) ||


in functions.mod.php
steht bei den berechtigungen "lay" sollte aber "mod" sein...

das wärs mal auf die schnelle was mir aufgefallen ist...
Ist auch gefixt. Ist krass, was du alles siehst :)

Verfasst: Mo 15. Dez 2003, 11:34
von emergence
timo hat geschrieben:
emergence hat geschrieben:in include.con_art_overview.php findet sich

Code: Alles auswählen

$tmp_rowid  = $idart."-".$idartlang."-".$lidcat."-".$idcatlang."-".$idcatart."-".$idlang;
das $lidcat sollte jedoch $idcat sein...
Bist du ganz sicher? lidcat ist die Kategorie des Artikels, die unter umständen anders sein kann als die aktuelle Kategorie (idcat) - zumindest weiß ich, daß das so seltsam gebaut wurde, weil sonst hätte man ja auch gleich die idcat nehmen können...
sagen wir mal so -> die zeile $lidcat = $lidcat; wurde aus dem cvs entfernt und ist sonst nirgendwo mehr in der datei vorhanden... alle 13 oder 14 $lidcat wurden durch $idcat ersetzt... somit nehme ich mal an, das es dort ebenfalls ersetzt werden sollte...

Verfasst: Mo 15. Dez 2003, 11:40
von timo
emergence hat geschrieben:
timo hat geschrieben:
emergence hat geschrieben:in include.con_art_overview.php findet sich

Code: Alles auswählen

$tmp_rowid  = $idart."-".$idartlang."-".$lidcat."-".$idcatlang."-".$idcatart."-".$idlang;
das $lidcat sollte jedoch $idcat sein...
Bist du ganz sicher? lidcat ist die Kategorie des Artikels, die unter umständen anders sein kann als die aktuelle Kategorie (idcat) - zumindest weiß ich, daß das so seltsam gebaut wurde, weil sonst hätte man ja auch gleich die idcat nehmen können...
sagen wir mal so -> die zeile $lidcat = $lidcat; wurde aus dem cvs entfernt und ist sonst nirgendwo mehr in der datei vorhanden... alle 13 oder 14 $lidcat wurden durch $idcat ersetzt... somit nehme ich mal an, das es dort ebenfalls ersetzt werden sollte...
Ich habe ehrlichg esagt nicht die geringste Ahnung - da aber $lidcat über

$lidcat = $sart["idcat"];

zugewiesen wird, sollte das auch kein Drama sein.

Verfasst: Mo 15. Dez 2003, 11:43
von emergence
timo hat geschrieben:Ist auch gefixt. Ist krass, was du alles siehst :)
na ja ich übernehme immer nur teile aus dem cvs die ich bei meiner testversion einbaue und anpasse - und da bei mir die änderungen farblich hervorgehoben werden zwischen meiner und eurer version sehe ich diese dinge sofort.

wie auch immer einen hab ich noch: include.mycontenido_lastarticles.php
wird die d.idaction mittels ziffer angesprochen (2x) ich hab mir oben folgendes in den code eingebaut:

Code: Alles auswählen

            $sql = "SELECT
                    idaction
                FROM
                    ".$cfg["tab"]["actions"]."
                WHERE
                    name = 'con_editart'";

        $db->query($sql);
        $db->next_record();

        $idaction = $db->f("idaction");
und anschließend die ziffern durch

Code: Alles auswählen

".$idaction."
ersetzt.

Verfasst: Mo 15. Dez 2003, 11:49
von timo
na ja ich übernehme immer nur teile aus dem cvs die ich bei meiner testversion einbaue und anpasse - und da bei mir die änderungen farblich hervorgehoben werden zwischen meiner und eurer version sehe ich diese dinge sofort.
Good Point.
wie auch immer einen hab ich noch: include.mycontenido_lastarticles.php
wird die d.idaction mittels ziffer angesprochen (2x) ich hab mir oben folgendes in den code eingebaut:
Gute Idee. Machs doch einfach über die $perm:

$idaction = $perm->getIDForAction("con_editart");

Verfasst: Mo 15. Dez 2003, 11:51
von emergence
timo hat geschrieben:Ich habe ehrlich gesagt nicht die geringste Ahnung - da aber $lidcat über

$lidcat = $sart["idcat"];

zugewiesen wird, sollte das auch kein Drama sein.
ach das hab ich gar nicht gesehen, da ich die sync routine momentan nicht mit übernehme... hast recht...

Verfasst: Mo 15. Dez 2003, 11:57
von emergence
timo hat geschrieben:Gute Idee. Machs doch einfach über die $perm:

$idaction = $perm->getIDForAction("con_editart");
och die steht da zur verfügung, ja dann... nehm ich sie...

ähm das bei visual edit zeigts mir aber nichts an obwohl das mit alt und title so nett im quellcode steht...

Verfasst: Mo 15. Dez 2003, 12:04
von timo
emergence hat geschrieben: ähm das bei visual edit zeigts mir aber nichts an obwohl das mit alt und title so nett im quellcode steht...
Komisch. Welcher Browser?

Verfasst: Mo 15. Dez 2003, 12:07
von emergence
ie 6.0.28 auf winme...

Verfasst: Mo 15. Dez 2003, 12:15
von timo
Ist ja wirr. Ich baue da noch die Container-Nummer davor :)

Der IE mag wohl kein Alt-Tag bei Select-Elementen...

Verfasst: Mo 15. Dez 2003, 12:30
von emergence
tja ein problem hab ich jetzt noch... bin gerade beim testen...

bei einem output modul wenn ich zB
<?php echo "any\n"; ?>
eingebe erhalte ich als ausgabe

anyn

ähm...

Verfasst: Mo 15. Dez 2003, 12:31
von timo
Ja, das Problem ist bekannt, ich bin aber noch nicht zum fixen gekommen. Liegt am addslashes/stripslashes-Handling.

Verfasst: Mo 15. Dez 2003, 12:47
von emergence
anscheinend liegt es am neuen container code

bei include.con_editcontent.php, und functions.con.php

Code: Alles auswählen

$code = preg_replace("/<container(.*)id=\"$value\"(.*)>/i", addslashes($output), stripslashes($code));
ohne diese zeile habe ich das problem nicht...

vielleicht hilfts dir ja...

Verfasst: Mo 15. Dez 2003, 12:58
von timo
ersetz das mal durch:

Code: Alles auswählen

$code = preg_replace("/<container(.*)id=\\\\\"$value\\\\\"(.*)>/i", $output, $code);
Nicht über die vielen Backslashes wundern :)