Startartikel Eigenschaft wird gelöscht

Beleuchtfix
Beiträge: 1082
Registriert: Di 22. Jul 2003, 10:14
Wohnort: Hessen
Kontaktdaten:

Startartikel Eigenschaft wird gelöscht

Beitrag von Beleuchtfix » Do 22. Jun 2006, 22:40

Wie haben bei dem Modul PSMetatags festgestellt, dass die Startartikel Eigenschaft gelöscht wird, wenn der User nicht berechtigt ist, diese Eigenschaft zu bearbeiten. Bug :?:

Siehe http://contenido.org/forum/viewtopic.ph ... highlight=

Ende der 2. Seite.
Gruß
Florian

emergence
Beiträge: 10641
Registriert: Mo 28. Jul 2003, 12:49
Wohnort: Austria
Kontaktdaten:

Beitrag von emergence » Fr 23. Jun 2006, 05:55

kann sein.. wenns ohne funktioniert ziemlich sicher fehler des plugins...

ähm, und jetzt ?
*** make your own tools (wishlist :: thx)

Beleuchtfix
Beiträge: 1082
Registriert: Di 22. Jul 2003, 10:14
Wohnort: Hessen
Kontaktdaten:

Beitrag von Beleuchtfix » Fr 23. Jun 2006, 10:17

Ohne Plugin hat vielleicht noch keiner ausgetestet.

Dann muss ich wohl mal einen Zugang ohne Plugin und mit entsprechenden Rechten einrichten und austesten.

Gruß
Florian

Beleuchtfix
Beiträge: 1082
Registriert: Di 22. Jul 2003, 10:14
Wohnort: Hessen
Kontaktdaten:

Beitrag von Beleuchtfix » Fr 23. Jun 2006, 14:38

Ich habe das Ganze jetzt mit einer fast leeren 4.6.8 Demo getestet.

Einzig gesetzten Rechte sind Artikeleigenschaften bearbeiten. Startartikel setzen ist nicht erlaubt.

Sobald die Artikeleigenschaften bearbeitet wurden, wird die Eigenschaft Startartikel gelöscht :!:

Also wohl doch ein Bug in Contenido.
Gruß
Florian

mvf
Beiträge: 1758
Registriert: Mo 1. Aug 2005, 00:35
Wohnort: in der schönen Hallertau, mitten im Hopfen
Kontaktdaten:

Beitrag von mvf » Fr 23. Jun 2006, 17:46

Beleuchtfix hat geschrieben:Sobald die Artikeleigenschaften bearbeitet wurden, wird die Eigenschaft Startartikel gelöscht :!:
kann ich bestätigen für 4.6.8.5 und die MR von stese vom 03.06., beide ohne psMetaTag Plugin!

denke auch das dies ein bug ist :?
Grüsse, Guido

"A common mistake that people make when trying to design something completely foolproof is to underestimate the ingenuity of complete fools."
Mostly Harmless - Douglas Adams

HerrB
Beiträge: 6935
Registriert: Do 22. Mai 2003, 12:44
Wohnort: Berlin
Kontaktdaten:

Beitrag von HerrB » Fr 23. Jun 2006, 22:57

Ja, ist einer, wurde auch schon mal irgendwo gepostet...

Gruß
HerrB
Bitte keine unaufgeforderten PMs oder E-Mails -> use da Forum!

Newsletter: V4.4.x | V4.6.0-15 (Module, Backend) | V4.6.22+
Standardartikelliste: V4.4.x | V4.6.x
http://www.contenido.org/forum/search.php | http://faq.contenido.org | http://www.communido.net

emergence
Beiträge: 10641
Registriert: Mo 28. Jul 2003, 12:49
Wohnort: Austria
Kontaktdaten:

Beitrag von emergence » Sa 24. Jun 2006, 01:17

entweder ist das ein bug in der
db: con_actions - con_saveart

oder in der functions.con.php
-> function conEditArt

das klingt für mich so das ganze komplett auseinander zu nehmen und am besten gleich neu zu schreiben... (meine güte -> das wird stunden kosten)
da kann man auch gleich die conEditFirstTime mit überarbeiten (zwei funktion machen was ähnliches aber nicht 100%ig ident...)

jetzt frag ich mal ganz blöd, hat jemand lust das zu finanzieren ?
*** make your own tools (wishlist :: thx)

m.wohlers
Beiträge: 93
Registriert: Di 25. Mai 2004, 16:11
Wohnort: Bad Aibling
Kontaktdaten:

Beitrag von m.wohlers » Sa 24. Jun 2006, 18:07

Hallo!
emergence hat geschrieben:jetzt frag ich mal ganz blöd, hat jemand lust das zu finanzieren ?
Also ich stimme für ein "Bounty-System", bei dem gewissen Features durch die Community finanziert werden können...

Wäre sofort mit einem zweistelligen Betrag dabei!

Mit schwarz-rot-goldenen Grüssen...
Michael Wohlers

emergence
Beiträge: 10641
Registriert: Mo 28. Jul 2003, 12:49
Wohnort: Austria
Kontaktdaten:

Beitrag von emergence » So 25. Jun 2006, 09:27

yep, wäre ne nette idee...
*** make your own tools (wishlist :: thx)

m.wohlers
Beiträge: 93
Registriert: Di 25. Mai 2004, 16:11
Wohnort: Bad Aibling
Kontaktdaten:

Beitrag von m.wohlers » So 25. Jun 2006, 21:31

Hallo,
emergence hat geschrieben:yep, wäre ne nette idee...
Wie heißt es so schön?
Du bist Deutschland!
8)

Also ich bin dabei, das müsste nur jemand organisieren (à la Communido)...

Mit freundlichen Grüssen,
Michael Wohlers

xmurrix
Beiträge: 3143
Registriert: Do 21. Okt 2004, 11:08
Wohnort: Augsburg
Kontaktdaten:

Beitrag von xmurrix » Di 27. Jun 2006, 13:26

Hallo zusammen,

habe das Problem mit dem eingrenzen können. In der Datei "contenido/includes/include.con_edit_form.php" steht in der Zeile 327 folgender Eintrag:

Code: Alles auswählen

$tmp_start = ($tmp_is_start == 0) ? '<input disabled="disabled" type="checkbox" name="" value="1">' : '<input disabled="disabled" type="checkbox" name="" value="1" checked="checked">';
Da der Name "is_start" der Checkbox nicht gesetzt wird, wenn der User keine Berechtigung hat, wird die Eigenschaft beim Speichern auf 0 gesetzt.

Die Änderung auf

Code: Alles auswählen

$tmp_start = ($tmp_is_start == 0) ? '<input disabled="disabled" type="checkbox" name="is_start" value="1">' : '<input disabled="disabled" type="checkbox" name="is_start" value="1" checked="checked">';
sollte das Problem beheben...

Grüße
xmurrix

emergence
Beiträge: 10641
Registriert: Mo 28. Jul 2003, 12:49
Wohnort: Austria
Kontaktdaten:

Beitrag von emergence » Fr 30. Jun 2006, 17:24

intressante variante...
bin mir da aber nicht sicher ob bei einem disabled="disabled" die gesetzten werte übertragen werden... (das muss nicht bei allen browsern ident sein...)

na wie auch immer, vielleicht kann ja einer testen und hier feedback geben (intressant obs bei firefox, ie, safari geht)
*** make your own tools (wishlist :: thx)

xmurrix
Beiträge: 3143
Registriert: Do 21. Okt 2004, 11:08
Wohnort: Augsburg
Kontaktdaten:

Beitrag von xmurrix » Mo 3. Jul 2006, 11:12

emergence hat geschrieben:bin mir da aber nicht sicher ob bei einem disabled="disabled" die gesetzten werte übertragen werden... (das muss nicht bei allen browsern ident sein...)

na wie auch immer, vielleicht kann ja einer testen und hier feedback geben (intressant obs bei firefox, ie, safari geht)
IE, Firefox, Opera, Netscape übertragen keine Daten von Formularfeldern, die das Attribut "disabled" haben. Also war mein Vorschlag für die Katz.

Die Idee, "readonly" anstatt "disabled" zu verwenden, ist auch keine Lösung, da dies bei einer Checkbox gar keinen Einfluss hat.

Eine schnelle Lösung wäre in der Datei 'include.con_edit_form.php' zusätzlich ein Hidden-Feld mit dem aktuellen $is_start-Wert hinzuzufügen, wenn der User kein Recht auf das Setzen der Startartikel Eigenschaft hat.

Aber in der DB (con_actions - con_saveart) sehe ich einen Fehler ab der Zeile 156:

Code: Alles auswählen

$sql = "SELECT * FROM ".$cfg["tab"]["cat_lang"]." WHERE idcat = '$idcat' AND idlang = '$lang' AND startidartlang != '0' ";
$db->query($sql);
if ( $db->next_record() )
{
    $tmp_startidartlang = $db->f('startidartlang');
    if ($idartlang == $tmp_startidartlang)
    {
        conMakeStart($tmp_idcatart, 0);
    }else
    {
        ; # do nothing
    }
    
}else
{
    conMakeStart($tmp_idcatart, 0);
}

Wenn $idartlang gleich $tmp_startidartlang ist, dann sollte auch die Startartikel Eigenschaft nicht auf '0' gesetzt werden, also:

Code: Alles auswählen

    $tmp_startidartlang = $db->f('startidartlang');
    if ($idartlang == $tmp_startidartlang)
    {
        ; # do nothing
    }else
    {
        conMakeStart($tmp_idcatart, 0);
    }
Was meint ihr?

Grüße
xmurrix

emergence
Beiträge: 10641
Registriert: Mo 28. Jul 2003, 12:49
Wohnort: Austria
Kontaktdaten:

Beitrag von emergence » Mo 3. Jul 2006, 18:50

Was meint ihr?
hmm... eine erste bereinigung der action hab ich ja mal vorgenommen...
hier zu finden -> http://www.contenido.org/forum/viewtopi ... 2039#82039
änderungen daran vorzunehmen ist vielleicht leichter... (könnte auch sein das ich obriges dort schon gekillt hab... weiss ich jetzt aber nicht mehr...)

die andere sache
der code mit is_start hat in der action con_saveart sowieso nichts verloren...
sollte in die funktion rein... und in der funktion findet sich auch ein have_perms aufruf... hat auch nix dort verloren... sollte in die action...

das führt aber wieder dazu das zeugs neu schreiben zu müssen...
(die mehrfachzuordung wird ja auch noch in der funktion erledigt...)

ist noch immer viel arbeit...
*** make your own tools (wishlist :: thx)

xmurrix
Beiträge: 3143
Registriert: Do 21. Okt 2004, 11:08
Wohnort: Augsburg
Kontaktdaten:

Beitrag von xmurrix » Mo 3. Jul 2006, 19:33

eine erste bereinigung der action hab ich ja mal vorgenommen...
hier zu finden -> http://www.contenido.org/forum/viewtopi ... 2039#82039
änderungen daran vorzunehmen ist vielleicht leichter... (könnte auch sein das ich obriges dort schon gekillt hab... weiss ich jetzt aber nicht mehr...)
Der Code sieht schon viel besser aus. Allerdings wird auch hier die Startartikel-Eigenschaft auf '0' gesetzt. Ab Zeile 71 den Code:

Code: Alles auswählen

                        $tmp_startidartlang = $db->f('startidartlang'); 
                        if ($idartlang == $tmp_startidartlang) 
                        { 
                            conMakeStart($tmp_idcatart, 0); 
                        } 
ersetzen mit

Code: Alles auswählen

                        $tmp_startidartlang = $db->f('startidartlang'); 
                        if ($idartlang != $tmp_startidartlang) 
                        { 
                            conMakeStart($tmp_idcatart, 0); 
                        } 
Wenigstens ist dann der Fehler behoben.

Der Sache mit der Arbeit an "con_saveart", "conEditFirstTime" und "conEditArt" stimme ich zu. Auch wenn ich mit Contenido sehr zufrieden bin, gibt es doch hier und da immer Möglichkeiten zur Verbessung/Optimierung.

Gruß
xmurrix

Gesperrt