Contrib.social

Option to show allowed terms as checkboxes

Open on Drupal.org →
Created on 27 October 2023, almost 2 years ago

Problem/Motivation

With a larger vocabulary where each user has multiple allowed terms selected in their user edit form if a site manager accidentally doesn't hold down the Ctrl or Cmd key when clicking on a new term they're adding, all the terms that were previously selected get lost. This is a nightmare for sites with lots of users and lots of terms.

Steps to reproduce

1. Have a vocabulary with many terms.
2. Edit the user to add a new term.
3. Accidentally click on the new term without holding the Ctrl/Cmd key.

Proposed resolution

Create an option to output the list of terms as Checkboxes so that each term is independently an on/off check and will not be affected by other terms.

✨ Feature request
Status

Active

Version

3.1

Component

Code

Created by

🇺🇸United States marty.true

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @marty.true
  • Comment almost 2 years ago →
    🇮🇳India sidharth_soman Bangalore

    I have introduced an option for choosing the checkboxes output in the module's config and also a conditional for outputting the terms in checkboxes in the user form.

    Please apply the patch and review.

  • Comment almost 2 years ago →
    🇺🇸United States marty.true

    The patch applied cleanly, and the new option works a treat! Thank you for addressing this issue — this option makes the module much more feasible now.

  • First commit to issue fork.
  • Merge request !66Issue #3397350 by sidharth_soman, vitaliyb98: Option to show allowed terms as checkboxes → (Open) created by Unnamed author
  • Comment about 2 months ago →
    🇺🇦Ukraine vitaliyb98

    Improved code provided in the patch.
    Fixed error with unsetting the checkbox:

    TypeError: Drupal\permissions_by_term\Service\AccessStorage::addTermPermissionsByUserIds(): Argument #2 ($termId) must be of type string, int given, called in /var/www/html/modules/permissions_by_term/permissions_by_term.module on line 339 in Drupal\permissions_by_term\Service\AccessStorage->addTermPermissionsByUserIds() (line 274 of /var/www/html/modules/permissions_by_term/src/Service/AccessStorage.php).

    Added functional tests for this functionality.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024