Currently this appears to only support 128 maxlength path name.
D7.22 appears to support 255.
This is a simple patch to increase the support here to the same.
It doesn't appear to have any negative effects.
The 'path' data has been reviewed and is serialized data.

CommentFileSizeAuthor
#4 views-page_display_path_maxlength_should_mirror-2025443-4.patch584 bytesAndrew Answer
#1 maxlength.patch581 byteskvanderw
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kvanderw’s picture

kvanderw CreditAttribution: kvanderw commented
FileSize
maxlength.patch581 bytes

forgot the patch.

kvanderw’s picture

kvanderw CreditAttribution: kvanderw commented
Issue summary: View changes

updated initial explanation.

Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews commented
Assigned: kvanderw » Unassigned
Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Needs reroll

The 6 year old patch to views_plugin_display_page.inc does not apply to the latest views 7.x-3.x-dev.

Checking patch plugins/views_plugin_display_page.inc...
error: while searching for:
	  '#field_prefix' => '<span dir="ltr">' . url(NULL, array('absolute' => TRUE)) . (variable_get('clean_url', 0) ? '' : '?q='),
	  '#field_suffix' => '</span>&lrm;',
	  '#attributes' => array('dir'=>'ltr'),
        );
        break;
      case 'menu':

error: patch failed: plugins/views_plugin_display_page.inc:315
error: plugins/views_plugin_display_page.inc: patch does not apply
Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews commented
Andrew Answer’s picture

Andrew Answer CreditAttribution: Andrew Answer as a volunteer commented
Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
views-page_display_path_maxlength_should_mirror-2025443-4.patch584 bytes

Patch rerolled.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Category: Task » Bug report
Status: Needs review » Reviewed & tested by the community
Parent issue: » #2960871: Plan for Views 7.x-3.23 release

Simple fix, confirmed that node paths can be 255.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Reviewed & tested by the community » Fixed

Committed. Thanks!

  • DamienMcKenna committed c2b1c57 on 7.x-3.x authored by kvanderw 
    Issue #2025443 by kvanderw, Andrew Answer, Chris Matthews, DamienMcKenna...

Status: Fixed » Closed (fixed)

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