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.Postponed on #2041885: Move HTTP basic authentication provider to a separate module.
Now that we have a new "Web Services" package for HTTP basic auth module we should use it for the other web service related modules as well.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | Capture.PNG | 9.22 KB | garphy |
| #5 | web-services-package-2092817-5.patch | 1.86 KB | klausi |
| #3 | WebServices.png | 27.13 KB | StuartJNCC |
| #1 | web-services-package-2092817-1.patch | 1.88 KB | klausi |
Comments
Comment #1
klausiWe can add a changelog entry here as well, while we are at it.
Comment #2
jibranI like this RTBC for me let's see what @Crell has to say about this.
Comment #3
StuartJNCC CreditAttribution: StuartJNCC commentedIf I apply the patch in #1, I get two groups called "WEB SERVICES" in the modules list:
The problem seems to be that in basic_auth.info.yml the line is:
package: Web serviceswhilst those added by the patch are:
+package: Web Servicesi.e. capital "S" on "Services". Easy to fix the patch, but should a minor case difference like this matter when the package names are forced into upper-case anyway before display?
Comment #4
webchickOh. That looks silly. :)
The standard is "Capitalize only the first word."
Comment #5
klausiRight, fixed that. Also moved the CHANGELOG fixes to the bottom.
Comment #6
garphy CreditAttribution: garphy commented#5 is OK for me.
Comment #7
garphy CreditAttribution: garphy commentedI think that's an easy one.
Comment #8
StuartJNCC CreditAttribution: StuartJNCC commentedOpened a new issue #2099299: Package labels in module .info.yml files are case sensitive for what I would consider to be the bug revealed here, that differences in case amongst the package labels cause the modules to be listed under separate, but identically named groups.
Comment #9
webchickCommitted and pushed to 8.x. Thanks!