Skip to content

Several ACL & Toolbar fixes for com_finder (Smart Search)#221

Closed
sanderpotjer wants to merge 2 commits intojoomla:masterfrom
sanderpotjer:finder-aclfixes
Closed

Several ACL & Toolbar fixes for com_finder (Smart Search)#221
sanderpotjer wants to merge 2 commits intojoomla:masterfrom
sanderpotjer:finder-aclfixes

Conversation

@sanderpotjer
Copy link
Copy Markdown
Member

This patch fixes several ACL and toolbar fixes for com_finder (Smart Search).

  1. Changed position of Option button in toolbar to next to the help button (so it is in line with all other components) for several views

  2. The Index button in the toolbar of Indexed Content view is now only visible when user is allowed for the core.create action instead of visible for user with access to the component.

  3. Purge button visible in toolbar of Indexed Content view when allowed for core.delete instead of core.edit.state (delete action, not a state change)

  4. Clickable state button is now checking for core.edit.state action setting instead of core.manage for several views

  5. The Edit button in the toolbar of the Search Filter view was visible when allowed for the core.create action, now checking for the core.edit action.

  6. "Create a filter" link (visible when no filters added yet) in the Search Filter table now only visible when allowed for the core.create action.

  7. When allowed for core.edit and denied for core.edit.state and editing a Search Filter you were able to change the status (published/unpublished). This is now only possible when allowed for core.edit.state action.

@sanderpotjer
Copy link
Copy Markdown
Member Author

@phproberto
Copy link
Copy Markdown
Contributor

I'm closing this as it's not mergeable and 1 year old. Feel free to open if you update it and are still interested in get it merged.

Thanks for contributing.

@phproberto phproberto closed this Oct 3, 2013
@sanderpotjer sanderpotjer deleted the finder-aclfixes branch December 23, 2013 13:47
wilsonge pushed a commit that referenced this pull request Jan 23, 2017
wilsonge pushed a commit to wilsonge/joomla-cms that referenced this pull request May 26, 2019
…ring

Fix language string in status-postinstallation
richard67 pushed a commit to richard67/joomla-cms that referenced this pull request Apr 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants