Closed (fixed)
Project:
LoginToboggan
Version:
7.x-1.2
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
20 Oct 2011 at 17:10 UTC
Updated:
23 Nov 2011 at 20:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
hunmonk commenteda setting for this seems like overkill. still, LT is making a pretty drastic move there that i don't think you can change with a theme override. so i'm toying with the idea in the attached patch as a reasonable solution. please try it out and let me know what you think.
Comment #2
isaac.niebeling commentedI'll try the patch in the near future -- at the moment, I've just commented out the unset line since I have to have this to the client this afternoon, but I'll definitely do it on my local copy and let you know. Just looking at it, it appears like it should work, though there's nothing in there yet for end users to actually be able to turn it on and off (without a quick bit of code or DB update).
Comment #3
hunmonk commentedthe proper way to use this new variable is to set in in settings.php, no db stuff necessary.
this is a very common technique for putting in configuration stuff that is overkill to add to the UI.
Comment #4
isaac.niebeling commentedAh, I hadn't thought of that. Thanks for the reminder.
Comment #5
hunmonk commentedcommitted #1 to 7.x-1.x-dev.