while attempting to add an aliase the noticed that the lenght of the input field is limited to 63 characters even though the max length in admin/settings/pathauto is set to 100.

Am I missing something?

Kind regards,

Frank

CommentFileSizeAuthor
#7 335385-path-maxlength-D5.patch1.82 KBasimmonds
#2 335385-path-maxlength-D7.patch2.47 KBDave Reid
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Title: admin/build/path/add field length is 63 even though max length is 100 » admin/build/path/add field length is 63 even though the database can hold 128
Project: Pathauto » Drupal core
Version: 5.x-2.3 » 7.x-dev
Component: User interface » path.module

That field is provided by the Drupal core path module (so I'm changing the component). The underlying database field is 128 characters so I think it makes sense to change the form field to be 128 characters as well.

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Active » Needs review
FileSize
335385-path-maxlength-D7.patch2.47 KB

Here you go. Also found it very weird the the path alias field on the node page was only limited to 250 chars...

alexanderpas’s picture

alexanderpas CreditAttribution: alexanderpas commented
Status: Needs review » Reviewed & tested by the community

nice find!

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Good catch. I've committed this to CVS HEAD and DRUPAL-6 so that should take care of it. Thanks! :)

Status: Fixed » Closed (fixed)

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

asimmonds’s picture

asimmonds CreditAttribution: asimmonds commented
Status: Closed (fixed) » Patch (to be ported)

Marked a 5.x issue #289656: Alias UI and DB size limit inconsistent as duplicate to this. I'll port the patch to 5.x-dev soonish...

asimmonds’s picture

asimmonds CreditAttribution: asimmonds commented
Version: 7.x-dev » 5.x-dev
Status: Patch (to be ported) » Needs review
FileSize
335385-path-maxlength-D5.patch1.82 KB

Backported patch for 5.x-dev

Dave Reid’s picture

Dave Reid
he/him
Primary language English
Location Nebraska USA
CreditAttribution: Dave Reid commented
Status: Needs review » Reviewed & tested by the community

Looks good and applies cleanly.

drumm’s picture

drumm
he/him
Location NY, US
CreditAttribution: drumm commented
Status: Reviewed & tested by the community » Fixed

Committed to 5.x.

Status: Fixed » Closed (fixed)

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