Creating drupal 7 version of this: #1004306: Optional breadcrumb link text

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Cybnext’s picture

Status: Active » Needs review
FileSize
2.31 KB

Here is a patch that implements this feature into D7.

Courtney.B’s picture

Applied patch, was able to go in and switch from Link Title to Node Title.

I think I did a no-no when patching, so I'm going to try it out again. Have reverted to pre-patch state and will edit this reply with the results.

>> EDIT:

Was able to access Admin > Node Hierarchy and switch between Node Link and Node Title. Afterwards, no dice.

Copied patch steps:

XXX@XXX.org [~/public_html/sites/all/modules/nodehierarchy]# patch -p0 < Feature-1162722-1.patch
can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git a/nodehierarchy.module b/nodehierarchy.module
|index 57ddea9..95a3fb5 100644
|--- a/nodehierarchy.module
|+++ b/nodehierarchy.module
--------------------------
File to patch: nodehierarchy.module
patching file nodehierarchy.module
Hunk #2 succeeded at 1317 (offset 1 line).

Error Message when accessing Node Parent:

PDOException: SQLSTATE[42S22]: Column not found: 1054 Unknown column 'node.title' in 'field list': SELECT ml.*, parent.*, parent.nid as pnid, node.title as node_title FROM {menu_links} ml LEFT JOIN {nodehierarchy_menu_links} parent on parent.mlid = ml.plid LEFT JOIN {nodehierarchy_menu_links} nh_ml on nh_ml.mlid = ml.mlid LEFT JOIN {node} on node.nid = nh_ml.nid WHERE ml.module = :module AND ml.mlid = :mlid; Array ( [:module] => nodehierarchy [:mlid] => 689 ) in _nodehierarchy_load_menu_link() (line 1320 of /home/librarye/public_html/sites/all/modules/nodehierarchy/nodehierarchy.module).

Reverting to pre-patch state until I receive update.

heyyo’s picture

Could we use this patch with last dev ?

ronan’s picture

Status: Needs review » Needs work

Nice idea. Tried it on head but it doesn't work as is, I'm afraid. The node title's not being loaded when the breadcrumb is generated.

Cybnext’s picture

Status: Needs work » Needs review
FileSize
2.3 KB

Been awhile, but here is a patch for this off of HEAD.