TYPO3 Core

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

I tested this feature and it works.

But to be honest: Isn't this a fix for a regression introduced with 6.0 instead of a feature?

In bigger websites with many editors, not every editor will have write access to all filemounts. But sometimes, editors still need to link to documents they can't manage.

In my opinion, this is an "enterprise feature" and it belongs to an LTS. I would suggest that this will be re-integrated into 6.2 instead of being postponed to 6.3.

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

I found out that the current approach is no proper replacement for the feature it was before.

When I apply the patch and then add a "read-only" mountpoint pointing to /fileadmin, every other filemount poining to a subfolder of /fileadmin, e.g. /fileadmin/foo and /fileadmin/bar, is also read-only.

With the old feature I had the possibility to create a read-only filemount at top level and still have writable filemounts for sub levels.

Personally, I have nothing again the "read-only" flag being applied in cascade to the sub-folders. This would be the behaviour, I would expect as End User. However, options.folderTree.altElementBrowserMountPoints is suppose to display some special Mount Points only in the context of the Element Browser. Those Mount Points are not meant to be displayed in the File Module. That is not supported in the current implementation. We could "better" initialize the file storage in the Element Browser IMO.

BTW, documentation must be updated.
http://docs.typo3.org/typo3cms/TSconfigReference/UserTsconfig/Options/Index.html

Either the option is obsolete with this patch and must be removed or it gets fixed.

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

Hi guys,

I already tried to reimplement options.folderTree.altElementBrowserMountPoints but without success :( You can have a look at the first patch sets in Gerrit where this approach is implemented.

I think the current solution fits much better in the current infrastructure but I see the problem with the subfolders and will look into it.

Cheers,
Alex

Hi Alexander

I like to approach since it makes more sense than a TypoScript property. But I don't see the use case for the current solution. It would only be usable for read-only contents provided by the admin for editors, which is quite an edge case IMO.

I'll think about it.

Lorenz

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

Can you please test with the new version? Please note the dependency on the patch for #55416.

When I apply the patch and then add a "read-only" mountpoint pointing to /fileadmin, every other filemount poining to a subfolder of /fileadmin, e.g. /fileadmin/foo and /fileadmin/bar, is also read-only.

This should now be the other way around.

When you add a read-only mountpoint to /fileadmin/ the subdirectories within fileadmin for which writable filemounts exist will now also be writable within the /fileadmin/ filemount.

I hope this makes more sense now :)

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/25720

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Steffen Ritter and I are currently working on a new implementation that doesn't add a database field. This way it can still go into 6.2. The patch is WIP, but here is what it will do:

It will re-integrate the TSconfig options.folderTree.altElementBrowserMountPoints in a backwards compatible way. In TYPO3 4.x, it worked like that:

options.folderTree.altElementBrowserMountPoints = documents,images

added a read-only filemount for folders fileadmin/documents and fileadmin/images to the Element Browser.

With FAL, an installation can have multiple storages and therefore the storage needs to be configured alongside the folder. To stay backwords compatible we assume that the old syntax refers to a folder in the default storage. So if the default storage is "fileadmin" on Local filesystem, the behaviour will be exactly the same, with the exception that it also displays the filemounts read-only in the File List Module.

The new syntax is:

options.folderTree.altElementBrowserMountPoints = 3:/test,documents,images,2:/

This will add:

• Storage 3/test
• [default storage]/documents
• [default storage]/images
• Storage 2:/

If you have overlapping storages, these are merged so that you don't have duplicates in the file browser.

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31075

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 14 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 15 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 16 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 17 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 18 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30685

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31496

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31500

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/31501