Problem/Motivation

There are 183 projects with uppercase characters. This causes problems that are listed in 3 other issues: #1189292: Let enforce lower case names for new projects , #200628: Problem with update of mixed case module names, #1189178: Capital letters in machine name confuses Drush (and me)

Proposed resolution

sdboyer will script a conversion and apply. He will also handle 301s for redirects.

Remaining tasks

Convert all URLs to all lowercase.

User interface changes

None.

API changes

None

Original report by redhatmatt

Converting the URLs to all lowercase. There are 183 projects with uppercase letters. sdboyer will script a conversion and apply with an update hook this will require around 4hr of time. 1hr will be spent taking care of 301s to redirect old to new. 1hr should also be applied for additional issues.

Comments

redhatmatt’s picture

StatusFileSize
new3.49 KB

list of projects to be converted.

senpai’s picture

Version: x.y.z » 7.x-2.x-dev
Assigned: redhatmatt » sdboyer
Priority: Normal » Major

Assigning to @sdboyer

marvil07’s picture

Status: Active » Needs review
StatusFileSize
new1.71 KB

The following patch does two things:

  • Adds a validation to notice if user is adding a project with uppercase letters(see if lowercased version is equal or not to original) via node form.
  • Adds an update function to convert project short name field value to lowercase. I guess the query only works on mysql, and for the default field storage.
    Originally I tried to use an entity field query as start point, but it's impossible to pass an expression to it, so I end up with this solution.

I'm not really sure if those are the only changes to make or if it's correct, but I would really appreciate a review :-)

marvil07’s picture

This patch now depends on #1189292-9: Let enforce lower case names for new projects , so here the only piece is the update function.

drumm’s picture

Assigned: sdboyer » marvil07
Status: Needs review » Fixed

Committed!

marvil07’s picture

Thanks!

BTW, some patches like the one here include metadata of git. You can apply them with git am and preserve the commit message ;-)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

added proper url filter