Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
The definition of "bypass advanced aggregation" permission doesn't follow the usual naming convention - the title should be started from a capital letter. See the screenshot and you'll see the difference - it could be a nightmare for the perfectionist.
Proposed resolution
Capitalize the first title letter to make it look great.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 3179767-6_after.png | 79.55 KB | kleiton_rodrigues |
| #6 | 3179767-6_patch_applied.png | 29.46 KB | kleiton_rodrigues |
| #6 | 3179767-6_before.png | 79.34 KB | kleiton_rodrigues |
| #3 | Before Patch.png | 140.7 KB | manojithape |
| #3 | After Patch.png | 95.17 KB | manojithape |
Comments
Comment #2
MatroskeenComment #3
manojithape CreditAttribution: manojithape at QED42 commentedVerified and tested patch#2. Patch applied successfully and looks good to me.
Testing Steps:
1. Install Advanced CSS/JS Aggregation
2. Go to this Extend-> Search "Advanced CSS/JS Aggregation"->Click on the "Permissions" link which presents under the module description
3. Verify "bypass advanced aggregation" text naming convention.
4. Apply patch successfully and now again verify text should display as "Bypass advanced aggregation".
Testing Results:
After applying the successful patch, text displayed as "Bypass advanced aggregation".
Comment #4
manojithape CreditAttribution: manojithape at QED42 commentedComment #5
manojithape CreditAttribution: manojithape at QED42 commentedComment #6
kleiton_rodrigues CreditAttribution: kleiton_rodrigues at CI&T commentedPatch #2 by @Matroskeen, applied successfully and looks good to me.
RTBC+1
Comment #7
thallesLooks good to me!
Comment #9
thallesThanks @all!