Registration and Change password urls have less size available than I needed.
This supposed a problem in my environment, but I thought that maybe I was using too long urls. However, I saw that the other url fields are longer than these, so I attach a patch for making them all equal.

CommentFileSizeAuthor
#1 urls-size-1131878-1.patch851 bytespenyaskito
PASSED: [[SimpleTest]]: [MySQL] 405 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more

Comments

penyaskito’s picture

FileSize
851 bytes
PASSED: [[SimpleTest]]: [MySQL] 405 pass(es). View

Attached a patch for 6.x-3.x, the branch that I used.

penyaskito’s picture

Status: Active » Needs review

This is already in 7.x-1.x, so no patch needed. Please review 6.x-3.x patch.

bfroehle’s picture

I thought the default #size is 60 and default #maxlength is 128. Is that not sufficient?

bfroehle’s picture

Status: Needs review » Fixed

Nonetheless, I've committed your patch which sets '#maxlength' to 255. In doing so, I also changed the maxlength of paths and hostnames to 255 to match the defaults in Drupal core, and reduced the maxlength of the 'CAS port' to 5 (since the maximum port number is 65536).

Thanks for your contribution! Committed to 6.x-3.x and 7.x-1.x.

Status: Fixed » Closed (fixed)

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

  • Commit 65ff879 on 7.x-1.x, 8.x-1.x authored by penyaskito, committed by bfroehle:
    Issue #1131878 - Registration and Change password urls have different...