? ? ? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
23 Feb 2017

Pull Request for Issue #14207

Summary of Changes

use folderlist and not text field

Testing Instructions

  • Try to setup a field.
  • confirm you can only select valid folders and only under the images folder for the gallery

Expected result

you can only select valid folders and only under the images folder for the gallery

Actual result

You can insert every path you want.

Documentation Changes Required

none

avatar zero-24 zero-24 - open - 23 Feb 2017
avatar zero-24 zero-24 - change - 23 Feb 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Feb 2017
Category Front End Plugins
avatar PhilETaylor
PhilETaylor - comment - 23 Feb 2017

but what if I wanted to use /media and not the /images folder as a root?

avatar zero-24
zero-24 - comment - 23 Feb 2017

Feel free to send a PR that handles both ? IIRC media should not contain the images you display to the user via a gallery this is why we have the images folder. Also this folder is not supported by the media manager or any other field in Joomla for storing images.

avatar PhilETaylor
PhilETaylor - comment - 23 Feb 2017

Im playing devils advocate... seen it before where "we" have assumed "site owners" will put images in a folder called "images" and then they put them in a folder called "media" instead ;-)

Im not in PR mode, Im in Bug Reporting mode. The coffee machine in the office is broken :-(

avatar zero-24
zero-24 - comment - 23 Feb 2017

Im not in PR mode, Im in Bug Reporting mode. The coffee machine in the office is broken :-(

hmm i can send PRs even without a coffee machine ;)

avatar PhilETaylor
PhilETaylor - comment - 23 Feb 2017

Lucky you, I cannot even function without coffee... I restored to buying instant coffee from next door office's vending machine...

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 24 Feb 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 24 Feb 2017

I have tested this item successfully on c509358


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

avatar infograf768 infograf768 - test_item - 25 Feb 2017 - Tested successfully
avatar infograf768
infograf768 - comment - 25 Feb 2017

I have tested this item successfully on c509358


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

avatar infograf768 infograf768 - change - 25 Feb 2017
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 25 Feb 2017

RTC. Thanks.


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

avatar rdeutz rdeutz - change - 26 Feb 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-02-26 12:40:52
Closed_By rdeutz
Labels Added: ? ? ?
avatar rdeutz rdeutz - close - 26 Feb 2017
avatar rdeutz rdeutz - merge - 26 Feb 2017
avatar brianteeman
brianteeman - comment - 8 Apr 2020

can someone delete the very funny spam comment

Add a Comment

Login with GitHub to post a comment