GBfilepicker + LisIt - kein Speichern für Benutzer möglich

kris29 · 17. April 2012 12:40

Ich nutze den GBfilepicker 1.3.2 in einem ListIt2-Modul und hab folgendes Problem. Nutzer ohne Adminrechte können kein Bild speichern. Der Upload funktioniert, aber nach der Auswahl wird nichts übernommen. Es wird anscheinend nichts in die DB eingetragen sad

Als Admin funktioniert alles.
Bug oder Einstellungssache - weiß jemand Rat?

CMSMS 1.10.3

mike-r · 17. April 2012 17:32

Kann ich nicht nachvollziehen mit

CMSMS 1.10.3
Listit2 1.1
GBfilepicker 1.3.1*

Einstellungen von GBFilepicker:

Dateifunktionen anzeigen: [x]
Verzeichniszugriff des Benutzers einschränken: [ ]

Benutzerrechte

Listit modify Dingens [x] ([x][x])
Use GBfilepicker [x]

Der Ordner, wo die Bilder reinkommen hat CHMOD 777 (wird in Deinem Fall egal sein, da es ja mit dem Admin funktioniert)

* kann ich grad nicht auf Deine Version updaten.

Beitrag geändert von mike-r (18. April 2012 00:56)

NaN · 17. April 2012 18:09

Mach mal mit debug und so wink

kris29 · 17. April 2012 20:42

Sorry wenn ich da mal dumm nachfragen muß: Den Debug-Code vom Frontend oder vom Backend untersuchen? Gibts da spezielle Abschnitte oder Errors nach denen ich Ausschau halten muß?

Klenkes · 17. April 2012 21:05

kris29 schrieb:

Den Debug-Code vom Frontend oder vom Backend untersuchen?
Gibts da spezielle Abschnitte oder Errors nach denen ich Ausschau halten muß?

In der config.php:

Hier klicken, um den Code zum Kopieren zu markieren

$config['debug'] = true;

Spuckt eventuell Fehler für's Backend und auch Fontend aus.

mike-r · 18. April 2012 00:55

kris29 schrieb:

Sorry wenn ich da mal dumm nachfragen muß [...]

Deine Einstellungen sind mit meinen identisch?

NaN · 18. April 2012 09:17

Gibt's da spezielle Abschnitte oder Errors nach denen ich Ausschau halten muß?

Naja, Fehler an sich würden schon mal reichen wink
Beim Debug-Modus, kommt beim Speichern doch so eine "Zwischenseite" (Debug enabled, redirecting disabled, follow this link ... usw.).
Gibt es auf dieser Seite Fehlermeldungen?
(irgendwas mit Error)
Du kannst Den Text auch kopieren und hier auswerten lassen.
(nicht hier posten, ist zu groß, lade es als textdatei irgendwo hoch und poste den link)

Oder einfach mal die Gegenprobe: Funktioniert es denn mit GBFilePicker 1.3.1?

kris29 · 18. April 2012 10:26

Den Debug-Code gibts hier: http://speedy.sh/QuQW5/debug.txt

Im CMS unter Systemprüfung sind alle Häkchen auf grün. Und wie gesagt, als Admin funzt alles, das Problem besteht nur für den Editor.

@mike-r: Bis auf die GBfilepicker-Version ist bei mir alles identisch zu deinen Angaben.

Die Gegenprobe mit GBfilepicker 1.3.1 hab ich noch nicht gemacht. Befürchte das ich dann wieder das Problem mit dem Datei-Upload bekomme (http://www.cmsmadesimple.de/forum/viewtopic.php?id=1993). Nach dem Update zu 1.3.2 war das behoben. Hm, bei mir ist wohl der Wurm drin, da scheinbar kein anderer das Problem so hat.

mike-r · 18. April 2012 11:42

Ich hab jetzt problemlos auf 1.3.2. geupdatet, geht alles prima.

dein Log lässt sich nicht herunterladen, warum packst Du das nicht auf http://nopaste.info/ oder Ähnliches?

NaN · 18. April 2012 11:46

mike-r schrieb:

dein Log lässt sich nicht herunterladen

Doch doch. Musst nur oben bei Download auf die Datei klicken wink
(Bringt aber nicht viel)

Also ich kann es ebenfalls nicht nachvollziehen.
CMSMS 1.10.3
Listit2 1.1
GBfilepicker 1.3.2

Welche Optionen verwendest Du im GBFilePicker?
Vielleicht liegt's am mime-Type?

Klenkes · 18. April 2012 14:09

Moment... jetzt muss ich mal blöd fragen.

Könnte es so sein, dass du einen neuen Eintrag machst,
- alle Daten eingibst,
- dann ein Bild hochlädst
- es aus dem Dropdown auswählst
- Speichern!
- Und es ist nichts gespeichert.

Aber... wenn du den Eintrag nochmal bearbeitest, das Bild nochmal auswählst und speicherst, WIRD es auch gespeichert?

Dann wäre es so wie bei mir. Auch als Admin.
(habe ich in meiner Testumgebung festgestellt)

CMSMS 1.10.3
ListIt2 1.1
GBFilepicker 1.3.2

Klenkes · 18. April 2012 14:20

Interessant.
Grade mal mit debug versucht:

Hier klicken, um den Code zum Kopieren zu markieren

...
Bis hier ist alles Okay. Dies ist das letzte Textfeld:
(mysql): SELECT value FROM actest_module_listit2_fieldval WHERE item_id = 9 AND fielddef_id = 5 LIMIT  1
Debug: (1.700097) - (usage: 9248632) - (peak: 9526152) 

Beim ersten Mal Speichern war dieser Eintrag des Bildes nicht da:
(mysql): INSERT INTO actest_module_listit2_fieldval (item_id, fielddef_id, value) VALUES (9, 5, 'images/kontakt/chevy_1.jpg')
Debug: (1.701765) - (usage: 9265472) - (peak: 9526152)

Erst beim zweiten mal Speichern wurde auch das Select ausgelesen.

NaN · 18. April 2012 14:44

Fehler gefunden.
Ist ein Problem beim GBFilePicker und dem Aktualisieren des Dropdowns.
Hat aber nichts damit zu tun, ob man Admin ist, oder nicht.

Macht mal den Test:

Artikel bearbeiten -> Dropdown aktualisieren -> ändern -> speichern.

Wird nicht klappen, weil beim Aktualisieren des Dropdowns der Name des Feldes (falsch) geändert und somit dessen Inhalt nicht gespeichert wird. (ListIt erkennt es dadurch einfach nicht mehr)

Das scheint ein Problem im Core zu sein. Denn der Link zum Aktualisieren des Dropdowns wird mit der API Funktion erstellt. Dabei wird der Name des Feldes übergeben. Die Core-Funktion macht dann aus name=customfield[1] -> name=customfield_1_
Somit ersetzt GBFP zwar das Dropdown mit dem aktualisierten, gibt ihm dabei aber einen falschen Namen. Muss mal schaun wo ich da bei GBFP ansetzen kann …

Klenkes · 18. April 2012 15:02

Stimmt.
Nach Update Dropdown wird die neue Auswahlauch nicht gespeichert.

NaN · 18. April 2012 15:28

Hm, da müssen ListIt und GBFP mal an einem Strang ziehen.
Ich kann den Fehler nicht beheben, ohne dass bei ListIt jene Zeilen Code, die versuchen das Problem mit Hilfe von str_replace etc. zu umgehen, geändert werden müssen.
Ich meine das hier in /modules/ListIt2/action.admin_edititem.php:

Hier klicken, um den Code zum Kopieren zu markieren

// GBFilePicker "Update dropdown" does not work if id contains "[]", strip it out

D.h. nicht der Core ändert da was, sondern ListIt, weil das mit dem Dropdown sonst nie funktioniert hätte. Aber mit dieser Änderung kommt der Rest von GBFP wiederum nicht klar.

NaN · 18. April 2012 15:39

Workaround:

Bei ListIt in der Datei /modules/ListIt2/action.admin_edititem.php diese Zeilen (464 - 478):

Hier klicken, um den Code zum Kopieren zu markieren

// GBFilePicker "Update dropdown" does not work if id contains "[]", strip it out
$savename = str_replace(array('[', ']'), '_', $name);

$gbfilepicker = $GBFilePicker->CreateFilePickerInput($GBFilePicker, $id, $name, $value, array(
	'media_type' => 'file',
	'file_extensions' => implode(',', $custom_flds[$custom_fld['fielddef_id']]['allow']),
	'dir' => $dir,
	'upload' => true,
	'exclude_prefix' => $custom_flds[$custom_fld['fielddef_id']]['exclude_prefix'],
	'mode' => ($custom_flds[$custom_fld['fielddef_id']]['filebrowser']?'browser':'dropdown')
));

$search       = 'name="'.$id.$savename;
$replace      = 'name="'.$id.$name;
$gbfilepicker = str_replace($search, $replace, $gbfilepicker);

folgendermaßen ändern:

Hier klicken, um den Code zum Kopieren zu markieren

$gbfilepicker = $GBFilePicker->CreateFilePickerInput($GBFilePicker, $id, $name, $value, array(
	'media_type' => 'file',
	'file_extensions' => implode(',', $custom_flds[$custom_fld['fielddef_id']]['allow']),
	'dir' => $dir,
	'upload' => true,
	'exclude_prefix' => $custom_flds[$custom_fld['fielddef_id']]['exclude_prefix'],
	'mode' => ($custom_flds[$custom_fld['fielddef_id']]['filebrowser']?'browser':'dropdown')
));

Und bei GBFilePicker in der Datei /modules/GBFilePicker/action.reloadDropdown.php Zeile 28:

Hier klicken, um den Code zum Kopieren zu markieren

$name = trim($params['name']);

so ändern:

Hier klicken, um den Code zum Kopieren zu markieren

$name = rawurldecode(trim($params['name']));

mike-r · 18. April 2012 15:50

NaN schrieb:

Macht mal den Test:
Artikel bearbeiten -> Dropdown aktualisieren -> ändern -> speichern.

Kann ich so nachvollziehen. (Hatte zum Testen die Einträge nicht bearbeitet, sondern nur erstellt)

Btw: Wozu braucht man das Aktualisieren-Feld eigentlich? Wird doch eh automagisch aktualisiert. (wo das hier besprochene Problem trotzdem auftritt)

Klenkes · 18. April 2012 16:00

NaN schrieb:

Workaround: [...]

Funktioniert hervorragend! Exzellent.

NaN · 18. April 2012 16:44

automagisch

lol

Wozu braucht man das Aktualisieren-Feld eigentlich?

Weil man ja auch parallel per FTP etwas hochladen könnte.
Die Upload-Funktion kann ja auch mal deaktiviert sein.
Wie bekommt man da dann sonst ein aktuelles Dropdown, ohne die komplette Seite neuladen zu müssen?
Außerdem brauche ich irgendeine Art Dummy, der die URL zum (magischen) Neuladen enthält (ich entnehme das z.Z. aus dem href-Attribut des Link). Könnte ich aber auch ändern.

Ich plane ja nun schon seit einer Weile eine bessere Template-Verwaltung einzubauen roll
Da kann man diesen Link auch entfernen, wenn er denn so stört.

kris29 · 19. April 2012 09:27

Danke für eure Antworten und den Workaround.

Forum für CMS/made simple

#1 17. April 2012 12:40

GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#2 17. April 2012 17:32

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#3 17. April 2012 18:09

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#4 17. April 2012 20:42

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#5 17. April 2012 21:05

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#6 18. April 2012 00:55

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#7 18. April 2012 09:17

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#8 18. April 2012 10:26

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#9 18. April 2012 11:42

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#10 18. April 2012 11:46

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#11 18. April 2012 14:09

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#12 18. April 2012 14:20

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#13 18. April 2012 14:44

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#14 18. April 2012 15:02

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#15 18. April 2012 15:28

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#16 18. April 2012 15:39

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#17 18. April 2012 15:50

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#18 18. April 2012 16:00

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#19 18. April 2012 16:44

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

#20 19. April 2012 09:27

Re: GBfilepicker + LisIt - kein Speichern für Benutzer möglich

Fußzeile des Forums