The Extend page has several things that could be better handled in the description of modules.
There are several references (4) to "your site" on the page. These are generally redundant and may be inaccurate. It could be someone else's site, a company's site etc. For consistency these should be eliminated.
In the descriptions most start with provides, enables or allows but there is little consistency in how these terms are used. For example both the PHP Filter and OpenID start with "allows" but the PHP Filter doesn't allow the execution of PHP code it enables the administrator to allow it with a permission. OpenID on the other hand is one of the few items that really does "allow" things on that page since there isn't another level of access control once the module is enabled. Some of the newer additions dispense with these but still could use help and standardization "Entity" says "Generic Entity Functionality" - very much an insider's game at this point as you must really know what Entities are and how they work in Drupal before this means anything.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | clean_up_module_description_text-1924996-1-before.png | 45.67 KB | andymartha |
| #4 | clean_up_module_description_text-1924996-1-after.png | 46.06 KB | andymartha |
| #1 | clean_up_module_description_text-1924996-1-.patch | 1.82 KB | jbrauer |
Comments
Comment #1
jbrauer commentedA patch for the missing "administrators" and your strings.
Comment #2
jbrauer commentedAlso while this patch follows the apparently intended "Allows administrators to" style we should carefully look at these strings to see if each of them should include the "actor" portion. Where things are controlled by permissions for example then it is really up to the site administrator to decide which actors can use a given functionality.
Comment #3
jbrauer commentedComment #4
andymartha commentedOn a clean installation of Drupal 8.x-dev from March 6, 2013, I can confirm the text problems described above. See screenshot.
After applying the clean_up_module_description_text-1924996-1-.patch, no side effects were found and proposed text changes for user extend page were made. See screenshot.
Maybe further text changes by jbrauer?
Comment #5
dries commented.info files are no more so this patch needs a re-roll. Thanks!
See #1793074: Convert .info files to YAML for details.
Comment #14
pameeela commentedThe grammar issue mentioned here and most of the modules (PHP filter, OpenID, Entity) are no longer relevant, and some of the descriptions have already changed.
I agree that the references to 'administrators' should be removed so I created a follow up task: #3223179: Remove references to 'administrators' in module descriptions.
Comment #15
pameeela commentedOops, did not mean to update the title.