Hello,

Great module and you helped me in the past on it, thanks. I can visualize a few small API improvements that would help (beyond the one already requested about per node-type enabling/disabling). After short discussion if these changes are acceptable, I could create a patch for this module.

  1. The JS rewrites all links on page load. This is both expensive and very general (as it evaluates all links on the page). The module could provide an API setting and custom module hook to allow a themer the ability to theme only certain page elements or fields (for the use case where only certain links on the page should be targeted). If so desired, the original behaviour is retained (I find for our marketing use-case we need only specific links to be targeted, and site-wide implementation is too much).
  2. add a component to the generated internal link: /external?url=http://example.com&referrer=my/referring/page OR /external?url=http://example.com&referrer=node/123
  3. add a variable in the template such that a fluid "back link" is generated (checked for url injection)

Comments

stongo’s picture

Sure, any patches would gladly be accepted. I'm just finishing a large project that has kept me busy for most of the summer, but I will be putting in some time for this module after the 12th of this coming month. I want to straighten out the git problems, run coder on the D7 release, and then release dev versions. Once the dev release is out, I would be more than happy to have some help and contributions :) It generally needs lots of code improvements, I agree.

stongo’s picture

jjchinquist, I have released a dev version for each drupal version. your help would still be appreciated :)

stongo’s picture

Status: Active » Closed (fixed)

Check 7.x-2.x-dev

jjchinquist’s picture

My apologies, I had other projects to work on and this one got pushed back. I will try to send feedback. - Jeremy