Startartikel Eigenschaft wird gelöscht
-
- Beiträge: 1082
- Registriert: Di 22. Jul 2003, 10:14
- Wohnort: Hessen
- Kontaktdaten:
Startartikel Eigenschaft wird gelöscht
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
Siehe http://contenido.org/forum/viewtopic.ph ... highlight=
Ende der 2. Seite.
Gruß
Florian
kann sein.. wenns ohne funktioniert ziemlich sicher fehler des plugins...
ähm, und jetzt ?
ähm, und jetzt ?
*** make your own tools (wishlist :: thx)
-
- Beiträge: 1082
- Registriert: Di 22. Jul 2003, 10:14
- Wohnort: Hessen
- Kontaktdaten:
-
- Beiträge: 1082
- Registriert: Di 22. Jul 2003, 10:14
- Wohnort: Hessen
- Kontaktdaten:
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
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
-
- Beiträge: 1758
- Registriert: Mo 1. Aug 2005, 00:35
- Wohnort: in der schönen Hallertau, mitten im Hopfen
- Kontaktdaten:
kann ich bestätigen für 4.6.8.5 und die MR von stese vom 03.06., beide ohne psMetaTag Plugin!Beleuchtfix hat geschrieben:Sobald die Artikeleigenschaften bearbeitet wurden, wird die Eigenschaft Startartikel gelöscht
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
"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
Ja, ist einer, wurde auch schon mal irgendwo gepostet...
Gruß
HerrB
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
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
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 ?
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)
Hallo!
Wäre sofort mit einem zweistelligen Betrag dabei!
Mit schwarz-rot-goldenen Grüssen...
Also ich stimme für ein "Bounty-System", bei dem gewissen Features durch die Community finanziert werden können...emergence hat geschrieben:jetzt frag ich mal ganz blöd, hat jemand lust das zu finanzieren ?
Wäre sofort mit einem zweistelligen Betrag dabei!
Mit schwarz-rot-goldenen Grüssen...
Michael Wohlers
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:
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
sollte das Problem beheben...
Grüße
xmurrix
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">';
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">';
Grüße
xmurrix
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)
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)
IE, Firefox, Opera, Netscape übertragen keine Daten von Formularfeldern, die das Attribut "disabled" haben. Also war mein Vorschlag für die Katz.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)
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);
}
Grüße
xmurrix
hmm... eine erste bereinigung der action hab ich ja mal vorgenommen...Was meint ihr?
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)
Der Code sieht schon viel besser aus. Allerdings wird auch hier die Startartikel-Eigenschaft auf '0' gesetzt. Ab Zeile 71 den Code: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...)
Code: Alles auswählen
$tmp_startidartlang = $db->f('startidartlang');
if ($idartlang == $tmp_startidartlang)
{
conMakeStart($tmp_idcatart, 0);
}
Code: Alles auswählen
$tmp_startidartlang = $db->f('startidartlang');
if ($idartlang != $tmp_startidartlang)
{
conMakeStart($tmp_idcatart, 0);
}
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