Needs work
Project:
Bad judgement
Version:
8.x-1.0
Component:
Crell
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 Nov 2010 at 05:51 UTC
Updated:
27 Mar 2011 at 15:57 UTC
Jump to comment: Most recent file
Comments
Comment #1
chx commentedDont forget to submit back the form.
Comment #2
chx commentedOh Dmitri wrote that already. Too baaaaaaad :p
Comment #3
johnalbinOk. Here's a patch.
Comment #4
carlos8f commentedLOL. +1
Comment #5
mbutcher commentedTested, and I can verify that this patch was "rolled" like no other.
Comment #6
IceCreamYou2 commentedWait, does this mean the module can never be disabled from the UI? What if someone installs a bad idea, realizes it's a bad idea, and then wants to remove it?
...oh, I get it.
Comment #7
dmitrig01 commented@JohnAlbin you are amazing
Comment #8
carlos8f commentedCan haz?
Comment #9
johnalbinFixed.
Comment #10
dave reidThe patch is missing critical functionality: it cannot commit itself.
Comment #11
wim leersThis is AWESOME :D
Comment #12
wizonesolutionsHas any body gotten the self comiting paches 2 werk? I ned this for a client. They insissted I use Bad Judgement on there sight.
Comment #13
danillonunes commentedIf there two modules, like Module A and Module B, where:
Module A depends on Module B
&&
Module B depends on Module A
Then when them both are enabled, they cannot be disabled due to inter-dependency (on the interface, the checkboxes are disabled).
I think that can be used to ensure that a bad decision cannot be rolled back.