About:
Allows a site administrator to easily manage large role tables.
Installation:
Install module as usual. Enable at "/admin/modules".
Usage:
Bulk Options: Add (/admin/config/people/bulk_role_management)
-------------------------------------------------------------
Add (one line per row) as many role entity terms as desired. Simply copy &
paste your list in the "Add bulk role entities" box, click "Submit". The role
entities will be added to the "role" table in the database.
Bulk Options: Delete/Edit (/admin/config/people/bulk_role_management/edit)
--------------------------------------------------------------------------
After adding new role entities, your role entities will be listed on this page
for editing or mass deletion. If you want to edit (which will allow you to
rename the role entity), click the "rename" link. To mass delete, select the
role entities you wish to delete and click "Delete Selected". The role entities
selected for deletion will be removed from your "role" table and "users_roles"
table. Finally, the "permissions" link will take you directly to the role
entity's permission page. If you create a large list of role entities, don't
use Drupal global permissions edit page (the one that shows all permissions
for all roles as it will be too large to manage). Everything will work fine,
just easier to edit one role entity's permissions table at a time.
The user's edit page (/user/[uid]/edit)
-----------------------------------
The user's edit page is changed from having the section called "roles" to
"My groups, memberships and roles" with a drill-down textbox. This drill-down
will allow you to quick search your role entities and jQuery will autofilter the
results allowing for fast role entity table selections.
Project page:
https://www.drupal.org/sandbox/xandermar/2585505
Git link:
git clone --branch 7.x-1.x http://git.drupal.org/sandbox/xandermar/2585505.git
Drupal core version 7.0
Comments
Comment #2
PA robot CreditAttribution: PA robot commentedThere are some errors reported by automated review tools, did you already check them? See http://pareview.sh/pareview/httpgitdrupalorgsandboxxandermar2585505git
Fixed the git clone URL in the issue summary for non-maintainer users.
We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)
Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).
I'm a robot and this is an automated message from Project Applications Scraper.
Comment #3
xandermar CreditAttribution: xandermar commentedFixes completed.
Comment #4
JKingsnorth CreditAttribution: JKingsnorth commentedComment #5
banviktor CreditAttribution: banviktor at CARD.com commentedFirst off I would suggest you to format the issue summary with HTML tags, not a simple copy-paste from README.txt.
Automated Review
No automated test cases found!
Manual Review
The starred items (*) are fairly big issues and warrant going back to Needs Work. Items marked with a plus sign (+) are important and should be addressed before a stable project release. The rest of the comments in the code walkthrough are recommendations.
This review uses the Project Application Review Template.
Comment #6
xandermar CreditAttribution: xandermar commentedUpdated the mandatory items marked "*" and most of the recommends. Please review and advise next steps. Thanks!
Comment #7
ttronslien CreditAttribution: ttronslien at Portage CyberTech commentedAutomated Review
Repeat issue (3 times?)
Translatable strings must not begin or end with white spaces, use placeholders with t() for variables
Concatenating translatable strings is not allowed, use placeholders instead and only one string literal
Concat operator must be surrounded by a single space
Expected one space after the comma, 0 found
Results: http://pareview.sh/pareview/httpgitdrupalorgsandboxxandermar2585505git
I think you can get rid of your error messages by doing something like:
drupal_set_message( t('Duplicate role entity: @role_name', array('@role_name' => $role->name)),'error');
Manual Review
Individual user account
No duplication
Master Branch
Licensing
3rd party assets/code
README.txt/README.md
Code long/complex enough for review
Secure code
Coding style & Drupal API usage
I noticed 1,2, 3 and 4from https://www.drupal.org/node/2585527#comment-10457533 is completed. WTG.
Recommendations
Suggestions for README
- Not sure if TOC is required, since I don't find it to be that long of a readme file :)
- Under requirements there is "none" listed however under troubleshooting it says, "Ensure you have the right version of jQuery running". Maybe add jQuery version to requirements?
Other
- You may want to add t() around text in your module. Not required, but I see you have been fairly consisten elsewhere (e.g. Line 25 in .module file.)
- Personally, I don't think adding links or functions into the Heading is both accessible or user friendly, contemplate adding tabs like many other modules in the future?
- bulk_role_management.js line 13 $("#user-admin-roles").prepend("
hello
"); Missing World ;) I'm sure it's for testing, but you may want to do something else with it.
- Use drupal behaviour over document.ready https://www.drupal.org/node/171213
Comment #8
xandermar CreditAttribution: xandermar commented@ttronslien - If I get the three issues you mentioned resolved, will this module be approved? I have a customer for whom I'm eagerly trying to get into production and am curious to next steps. I guess what I'm trying to say is that I'd really like to get this module released and usable by the community (out of sandbox) as I have many others that I will be adding very soon and want to make sure I do everything that is required as soon as possible.
Any advice is greatly welcomed. Thanks!
Comment #9
ttronslien CreditAttribution: ttronslien at Portage CyberTech commentedTo be honest, I have no idea at what point the module can be approved. I am brand new at this. I'm trying to sift through the documentation to actually get a clear sense of at what stage in the review process an application can be moved forward to the next stage. In the end though, it is a git administrator (and that is not me) that will give you the final approval. As the documentation for modules to reach project status suggests, it can take up to a year to have a module approved. I have however heard that is helps moving the process along if you yourself is participating in code reviews.
A module does not have to be approved by the community for you to use it with your clients though, although I do see the benefits and the credibility it provides.
Comment #10
xandermar CreditAttribution: xandermar commented@ttronslien - Thanks for the feedback! I will continue to dig in and help with code reviews :)
Comment #11
xandermar CreditAttribution: xandermar commentedComplete and ready for review.
Comment #12
xandermar CreditAttribution: xandermar commentedComment #13
xandermar CreditAttribution: xandermar commentedBump
Comment #14
apadernoThank you for your contribution!
I updated your account so you can promote this to a full project and also create new projects as either a sandbox or a "full" project.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on IRC in #drupal-contribute. So, come hang out and stay involved!
Thank you, also, for your patience with the review process. Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.
Thanks go also to the dedicated reviewer(s) as well.
Comment #15
xandermar CreditAttribution: xandermar commentedThank you!