-
-
Notifications
You must be signed in to change notification settings - Fork 158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Correctly show the file tree if the widget is restricted to a path #7180
base: 4.13
Are you sure you want to change the base?
Conversation
The problem is solved by the fix so far... However, the breadcrumb is not displayed in the popup - but I think that's better for usability. The breadcrumb only appears in the popup if I have clicked on a folder there or I accidentally clicked on exactly the same folder in the file manager. |
That is intended (see #6412 (comment)). |
Die Anzeige der Breadcrumb sollte man nochmal überdenken. Mit der jetzigen Implementierung erscheint die Breadcrumb im Popup nicht - außer ich habe vorher im Dateimanager zufällig den selben Pfad angeklickt, klicke den selben Pfad im Popup an oder einen Pfad innerhalb meiner (Vor-)Auswahl. Die Angabe des Pfades soll ja lediglich eine Hilfe sein, im richtigen Ordner zu landen und keine Restriktion auf einen Ordnerzugriff. Wenn man als Vorauswahl ... aber wenn das mit der Breadcrumb so sein soll - warum auch immer (??!) - dann bitte die Sachen so übernehmen. |
I don't think that's actually the case. This is a feature of the old and deprecated contao/core-bundle/src/Resources/contao/widgets/FileSelector.php Lines 156 to 205 in eb6c497
|
Wenn |
genau das was isch geschrieben habe... wenn der Pfad z.B. auf |
Wir haben das gerade kurz diskutiert und sind der Meinung, dass Es handelt sich um ein "custom root directory" und nicht um eine Vorauswahl für das Breadcrumb-Menü, also ist die Implementierung in diesem PR korrekt. Es ist aber ein Bug, dass man über die Suche aus dem festgelegten Pfad ausbrechen kann. (Nur ein Bug und keine Sicherheitslücke, da |
Fixed in 31a0ecf. Probably best viewed without whitespace changes. |
Fixes #6412
@zonky2 Can you please check if the changes solve your problem?