Footer union link gets the PINES link forced after user login.

[Anoop Atre]

Evergreen 1.6.0.6

Footer union link gets the PINES link forced after user login using the "default" skin.

This is because the "default" skin uses id "eg_gal_link" which forces the PINES link on login. It's a simple fix, change the id from "eg_gal_link" to "classic_link" which is used by the other footer links. Attached is a patch against trunk.

[Dan Scott (denials)]

To be honest, I don't understand why we would make this particular change in the default skin.

  * the link already has a class of "classic_link"; setting the id to "classic_link" seems strange
  * anyone running Evergreen in production will need to modify the skin, including the footer, and should blow this away if they're not in Georgia

However, I could see an argument for removing the link, and the corresponding buildEGGalLink() function in Open-ILS/web/opac/skin/default/js/sidebar.js, entirely, as it does add a small amount of attachEvt() overhead that should exist only in the PINES skin.

Chris, if you're reading this, would you be averse to pushing this functionality into the PINES-examples SVN repository?

[Jason Stephenson (jstephenson)]

Should this one be won't fix?