D5 backport?

toemaz - November 19, 2007 - 13:02
Project:Clickpath
Version:6.x-1.0
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs review
Description

Nice module! Thank you for releasing it.

Is there a D5.x version available? If not, are you planning to backport it?

#1

budda - November 26, 2007 - 11:45

Me too.

#2

eaton - December 12, 2007 - 04:46

I don't have any plans to, at least right now, but if someone took a stab at it I'd happily release it. The big trick would be the breadcrumb hijacking -- there's no good place that I can think of off the top of my head where the breadcrumb hijacking would work cleanly in D5. Any thoughts?

#3

Christefano - December 12, 2007 - 11:56

Please pardon me since I haven't even installed Clickpath yet, but I'm confused by the term "override" in the "Override breadcrumb trail" setting. By "breadcrumb hijacking" (said in #2), do you mean inserting Clickpath's own breadcrumbs (which more or less replicates the browser's back history) or simply hiding the breadcrumb trail?

#4

vacilando - April 28, 2008 - 11:53

+1 for a D5 backport, please.

#5

eaton - April 28, 2008 - 12:52
Status:active» won't fix

The method used to manipulate the breadcrumbs on all pages is not available in Drupal 6: this task would need to be done at the theme level, not the module level, in D5.

#6

a_c_m - July 18, 2008 - 10:13
Version:6.x-1.x-dev» 6.x-1.0
Status:won't fix» active

Attached is a simple backport of this module for D5.

To use, just install as you would normally, then in your theme (probably page.tpl.php) instead of echo'ing $breadcrumbs echo clickpath_get_breadcrumbs() appropriately themed, in the case of a phptemplate them that might be :

<?php
 
echo phptemplate_breadcrumb(clickpath_get_breadcrumbs());
?>

I've only breifly tested this backport and only with the breadcrumb replacement functionality, not block (but it might work).

Please review and comment.

a_c_m

AttachmentSize
clickpath.module.txt 5.7 KB

#7

a_c_m - July 18, 2008 - 10:14
Category:support request» feature request
Status:active» needs review

#8

vacilando - July 18, 2008 - 10:47

For D5 see also http://drupal.org/project/trailscout I think the functionality is very similar to D6 clickpath.

#9

a_c_m - July 18, 2008 - 14:50

Thanks, looks interesting but the back port works just fine for me :)

#10

coltrane - August 4, 2008 - 22:13

#11

christefano - August 4, 2008 - 22:17
Status:needs review» needs work

Please make a patch. Thanks!

http://drupal.org/patch/create

#12

a_c_m - August 5, 2008 - 06:06

strictly speaking its not a patch, its a whole different version!
The code i wrote works, isnt that enough for testing/review purposes?

I assumed a fully working file would be more accessible to most people for testing, once its confirmed it works, i will create a patch if your unwilling/unable to.

#13

a_c_m - August 5, 2008 - 06:05
Status:needs work» needs review

#14

coltrane - March 21, 2009 - 20:26

I'm sorry but Drupal 5 is far off my radar so testing and committing this is a low priority. If I get some positive reviews and demand I may commit.

 
 

Drupal is a registered trademark of Dries Buytaert.