Skip to content
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

[5.3] Accesslevel field - Fancy Select #44691

Open
wants to merge 1 commit into
base: 5.3-dev
Choose a base branch
from

Conversation

pe7er
Copy link
Contributor

@pe7er pe7er commented Jan 7, 2025

This PR adds the fancy select functionality to Joomla's core ''Access'' fields.

Summary of Changes

Joomla is suitable for large websites. On such large websites, you can have many User Groups and Viewing Access Levels. When there are many Viewing Access Levels, assigning content to the right Access Level is hard. This PR adds a fancy select feature to the Access field in the back-end forms to make it easier to select the right Access Level.

Testing Instructions

Add a lot of Viewing Access Levels to the website, e.g. using this SQL query:

INSERT INTO `#__viewlevels` (`id`, `title`, `ordering`, `rules`) VALUES
(7, 'Marketing', 5, '[]'),
(8, 'Legal department', 6, '[]'),
(9, 'Research & Development', 7, '[]'),
(10, 'Finance', 8, '[]'),
(11, 'Sales', 9, '[]'),
(12, 'Customer Service', 10, '[]'),
(13, 'Human Resources', 11, '[]'),
(14, 'IT', 12, '[]'),
(15, 'Maintenance department', 13, '[]'),
(16, 'Quality management', 14, '[]'),
(17, 'Operations management', 15, '[]'),
(18, 'Performance management', 16, '[]'),
(19, 'Account department', 17, '[]'),
(20, 'Purchasing', 18, '[]'),
(21, 'Communications', 19, '[]');

Check the Access field in the forms when you create a new item or edit an existing one on these pages:

  • Articles
    /administrator/index.php?option=com_content&view=articles
  • Articles: Categories
    /administrator/index.php?option=com_categories&view=categories&extension=com_content
  • Articles: Fields
    /administrator/index.php?option=com_fields&view=fields&context=com_content.article
  • Articles: Field Groups
    /administrator/index.php?option=com_fields&view=groups&context=com_content.article
  • Modules (Site)
    /administrator/index.php?option=com_modules&view=modules&client_id=0
  • Modules (Administrator)
    /administrator/index.php?option=com_modules&view=modules&client_id=1
  • Menus: All Menu Items
    /administrator/index.php?option=com_menus&view=items&menutype=
  • Contacts
    /administrator/index.php?option=com_contact&view=contacts
  • News Feeds
    /administrator/index.php?option=com_newsfeeds&view=newsfeeds
  • Tags
    /administrator/index.php?option=com_tags&view=tags
  • lugins
    /administrator/index.php?option=com_plugins&view=plugins
  • Guided Tours
    /administrator/index.php?option=com_guidedtours&view=tours
  • Languages: Content
    /administrator/index.php?option=com_languages&view=languages
  • Global Configuration (Default Access Level)
    /administrator/index.php?option=com_config

Actual result BEFORE applying this Pull Request

Go to Content > Articles /administrator/index.php?option=com_content&view=articles > Edit or New
The Access field is a large dropdown list with all Viewing Access Levels ordered by ordering.

access-before

Expected result AFTER applying this Pull Request

Go to Content > Articles /administrator/index.php?option=com_content&view=articles > Edit or New
The Access field is a smaller dropdownlist with the advanced searchable options feature.
Type a part of a name to filter the options.

access-after

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@brianteeman
Copy link
Contributor

brianteeman commented Jan 7, 2025

If it is decided that this PR is a good idea then the same changes need to be made to all other places that the access level field is displayed as a list eg in batch views and list filters

image

image

@fgsw
Copy link

fgsw commented Jan 8, 2025

@pe7er Should the Pull Request get a test as continuous-integration/drone/pr — Build is failing?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants