hello drupal admins...

i would like to submit a module to the drupal community which i feel helps fill a hole which i have seen as a request from some of the clients i have worked with in the past. my module's name is "jump links" and it provides a way to add random, miscellaneous links on a per-page basis. It generates a single block of these links which can be placed in any region on your site.

project link: http://drupal.org/sandbox/digitalsweetspot/1142956

users have an admin page where they can add/edit/delete their jump links. when adding or editing a jump link, the following form fields are available:

  • Display name: the name of the link as it is displayed to the user
  • Destination URL: the url which the user will be taken to upon clicking the link
  • Page(s) displayed on: a listing of pages which will be used to determine whether or not a Jump Link will be shown in the block

it takes some functionality from drupal's block module, but applies in in reverse. rather than telling the block which pages it is allowed to be viewed on, this module lets the jump links themselves determine which page they will be listed on, base on the "page(s) displayed on" data for that link.

i had been searching for other modules which do this, but was unable to find one. sure, there are other modules that provide simple redirect functionality, but nothing that acts like quite like this. this module acts like a menu system, but is much more flexible with the nature in how it outputs links. rather than building a menu, applying that menu to a block/region, and then hard-setting that block to only display on certain pages, which could ultimately cause you to have to create many, many menus and blocks...all of that functionality is achieve within one single block, whose nature is to be available at all times on every page, and let the list of jump links determine which pages this block will appear on.

while developing this module, i have received great feedback from other developers i have bounced this off of. i think this is a different way to approach adding links to a page, and one which the drupal community could greatly utilize.

i want to thank you in advance for your time and consideration. i look forward to hearing any feedback you may have.

cheers!
jon hopewell
(digitalsweetspot)

Comments

digitalsweetspot’s picture

Component: new project application » module

bump. just checking in on this. any feedback is much appreciated.

cheers!
digitalsweetspot

digitalsweetspot’s picture

bump. hello out there in drupal land...anyone home? could a drupal admin please take a look at this? if there is something that i have missed, please let me know what it is so i can get this module approved. i really do feel there is a need for something like this. if the issue is that there is no version for D7...please let me know and I will work on this.

any feedback would be welcomed.

cheers!
digitalsweetspot

svendecabooter’s picture

Priority: Normal » Critical

Hi,

You haven't done anything wrong. We're just battling with a long backlog of applications :(

Marking as critical as per http://drupal.org/node/894256
Sorry for the long wait. I'll try to review later on if I find more time. Hopefully someone else will step up as well in the meanwhile...

digitalsweetspot’s picture

hi svnedecabooter...

thanks for responding to this. odd i never got an email stating that there was a response, but c'est la vie! totally understand about the backlog...just wanted to make sure this was in the pipeline as i do think it could be a handy module.

it has been over a month since you last responded and marked this "critical"...has then been any headway?

please keep me posted. thanks again!

cheers!
-jon-
aka: digitalsweetspot

davidhernandez’s picture

Status: Needs review » Needs work

Looking this over now. You've been extraordinarily patient. On behalf of everyone, I'm sorry for the delay, but hopefully we can get this done quickly.

A few minor things:

* @file needs to be on its own line in the comment. See http://drupal.org/node/1354#files
* Make sure to use t() with the text you send to drupal_set_message(). You have several cases where it is missing.
* The main admin/content/jump_links element in hook_menu is missing a description. The description is what shows on the /admin page and a few other places.

Not so minor:

* There are improperly formed links. For example, in jump_links_list_page(), the 'No Jump Links exist' link is created using base_path(). You should use l(), instead. base_path() is better for creating real paths to files. Use l(), because it will account for whether or not the site is using clean urls. With clean urls off, this link breaks. The edit links for each jump link also break.
* This doesn't not work, in general, with clean urls off. This might be related to the previous comment, as it looks like you are basing the check directly on the requested url. request_uri() will return a url with '?q=' in it. The only way to make it work is to configure the jump link with the '?q=', like '?q=node1'. That wouldn't be good, from a usability standpoint.

misc’s picture

The applicant has been contacted to ask if the application is abandoned.

digitalsweetspot’s picture

The applicant has received your email and responded requesting that this project not be marked as abandoned, as he still plans on working out the issues listed in prior comments, as well as working on a D7 version too. :)

Thanks for your patience guys...I will try to attack this as soon as I can!

misc’s picture

Could we mark this application as postponed until you got the time to work on it?

misc’s picture

Status: Needs work » Postponed
klausi’s picture

Status: Postponed » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.