Option to disable glossary links

Hmm, interesting. I'd like to have others comment on this.

BTW, when you provide a patch, it is generally okay to assign the issue to yourself.

Hmm, now that a vocabulary is no longer required (http://drupal.org/node/233771), doesn't that pretty much take care of "do nothing?"

I'd appreciate the ability to have this feature, but I think the patch needs work. I think it doesn't reflect recent changes in the module.

I started to put it in and that other issue side-tracked me. Sorry. If you can re-do the patch, please do it against whichever release has the later date (usually -dev) as that will be most like the code I'm using on my development site.

I have got this working on the 5.x version and will hopefully port the changes to 6.x tomorrow.

My implementation is a bit different from yours:

1. I've changed the "absolute link" setting to "link style" with three choices: "absolute," "normal," and "none."
2. The link style setting is per input format, so one input format can produce links while another does not.
3. Disabling the related terms link is a new setting on the "General" settings page and will either produce just text, or links per the "Clicking on a link will" setting (the current method).
4. I've provided glossary_update_5103 (and whatever the 6.x number is) to convert the current "glossary_absolute_#' variables to the new "glossary_link_#" variables (with logging of the change). So make sure you run update.php after installing the new version.

Fix committed to both -dev versions.

Thanks for your hard work on this. Unfortunately, it's not working for me right now. Whenever I choose Link Type "none" and save my settings, it automatically gets reset to Link Type "normal" (when I view the settings page again, the selection "normal is chosen). So right now I can't take advantage of this feature.

Drat, one of the 5.x->6.x upgrade things snuck past me. I committed this to the -dev version.

Line 439 should be variable_set($link_name, $form_state['values'][$link_name]);

Great. It works perfectly now, thanks. May I ask why the glossary.admin.inc file is listed as a dead file in CVS? Took me a little while to figure that out.

Good question. I have no idea. I don't even know what "dead file" means. http://drupal.org/node/255386

From a site I found through Google:

Next, lets try removing a file so it no longer shows up when the package is checked out. The file "DeadFile" is intended for this.

So I'd say it's a mistake.

Well, I am not a CVS expert by any stretch of the imagination. I have no idea why it got marked dead since it is very much an active file, as is "empty.gif". The only thing I can think of is that the 5.x branch doesn't have the admin file.

I wonder if it's because I don't use the HEAD branch.