The links under /project should be in lower case.
For example, in Drupal they should be http://drupal.org/project/modules, http://drupal.org/project/themes, http://drupal.org/project/theme+engines.

-- Kiam@AVPnet

CommentFileSizeAuthor
#2 project_module.patch3.56 KBavpaderno

Comments

aclight’s picture

aclight commented
Project: Project » Drupal.org site moderators
Version: 5.x-1.2 »
Component: Projects » Other
Priority: Normal » Minor

Those links are created based on the taxonomy terms themselves, which are capitalized on drupal.org. I probably wouldn't be in favor of making the terms themselves lowercase, but I think this queue is the appropriate place to debate such a thing.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Project: Drupal.org site moderators » Project
Version: » 5.x-1.2
Component: Other » Projects
Priority: Minor » Normal
Status: Active » Needs review
StatusFileSize
new project_module.patch3.56 KB

The taxonomy terms are fine to have the first character in upper case, but it's still possible to have the links all in lower case, like the patch does.

Mine is just a proposal.

-- Kiam@AVPnet

dww’s picture

dww
we/he/they
commented
Status: Needs review » Closed (duplicate)

Thanks for the patch, however:

a) this patch itself has code-style problems
b) please don't fix unrelated problems in a patch (e.g. the existing code-style problems in the nearby menu item definitions).
c) + 'title' => $term->name, is an XSS vulnerability for the term name (minor, but we still don't do that).
d) nothing here deals with the link rot problem. we should have something that redirects from /project/Modules/* to /project/modules/*.
e) This is duplicate with #89777: Inconsistent capitalization of URLs for project/[Type] pages

So, please don't reply here. Instead, post a new patch over at #89777. Thanks!