? ? Pending

User tests: Successful: Unsuccessful:

avatar Fedik
Fedik
6 Dec 2020

Pull Request for Issue from comment #26750 (comment) .

Summary of Changes

Do not allow User to change ordering in the plugin parameters.
local-0:/, local-1:/ ... local-x:/ always should stay in original order.

I more like idea with named prefix <provider>-<account>, as @dgrammatiko suggested in #26750 (comment).
However this will not work, I not checked in deep but it seems that whole thing hardcoded to use <account> as integer.

Also I have fixed default value.

Testing Instructions

Apply path.
Go to Plugins: FileSystem - Local.
Add couple Directories, and try change their order.

Actual result BEFORE applying this Pull Request

You can change ordering

Expected result AFTER applying this Pull Request

You cannot change ordering

Documentation Changes Required

none

avatar Fedik Fedik - open - 6 Dec 2020
avatar Fedik Fedik - change - 6 Dec 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Dec 2020
Category Front End Plugins
avatar dgrammatiko
dgrammatiko - comment - 7 Dec 2020

@laoneo can you have a look at this one?

avatar laoneo
laoneo - comment - 16 Dec 2020

I didn't test, but looks fine for me...

avatar adj9
adj9 - comment - 19 Dec 2020

I have created folders with PR applied, but they remain fixed.
I don't know how to vary the order.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31599.

avatar ceford
ceford - comment - 3 Jan 2021

I have tested this item successfully on f53a54f

I can confirm the pull request removes the ordering thingy. But:

Can someone explain why this matters and how this plugin is supposed to be used? The Help screen is rubbish - I will fix it when I know more.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31599.

avatar ceford ceford - test_item - 3 Jan 2021 - Tested successfully
avatar Fedik
Fedik - comment - 3 Jan 2021

Can someone explain why this matters

It a bit tricky, as whole Adapters thing ?
There a bit #26750 (comment)
User can add some own folders here (many of them), that the system will see like 0, 1, 2 ... n (it is actual a Row indexes). If User change ordering, then association "Row Index" => "Row value" also will change, that will break previous stored values.

how this plugin is supposed to be used?

That a good question ?
It something for Media manager. I not searched, but I hope there a documentation.

avatar gostn
gostn - comment - 10 Jan 2021

I have tested this item successfully on f53a54f


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31599.

avatar gostn gostn - test_item - 10 Jan 2021 - Tested successfully
avatar ceford
ceford - comment - 10 Jan 2021

I have changed the Media Help screen to read as follows:

LOCAL Panel

This area shows the structure of the local folders of your site, by default the images folder.
If you wish to keep images and documents in separate local folders: first create a folder in
the root of your site, say files, then go to the Filesystem - Local plugin and add files under
Select directories. Back in the Media component you will see images and files as separate
items in the Local panel.

Which means that the Media Options descriptors are incorrect.

It will take a while for the change to pass through the translation system.

avatar richard67 richard67 - change - 13 Jan 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 13 Jan 2021

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31599.

avatar richard67 richard67 - change - 13 Jan 2021
Labels Added: ? ?
avatar wilsonge wilsonge - change - 23 Jan 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-01-23 20:46:23
Closed_By wilsonge
Labels
avatar wilsonge wilsonge - close - 23 Jan 2021
avatar wilsonge wilsonge - merge - 23 Jan 2021
avatar wilsonge
wilsonge - comment - 23 Jan 2021

Thanks!

Add a Comment

Login with GitHub to post a comment