yes, but we encourage our users to write COMPONENTS. so if i want my component to do something different when it is not enabled i have to do isEnabled()==false||isEnabledAllowed()==false to check for when the component is disabled.
and my point was that a lot of our users prob forget the isenabledallowed() check -igor On 11/4/07, Johan Compagner <[EMAIL PROTECTED]> wrote: > But we as the framework don't forget it ! > If a developer overwrites isVisible() now > then no mather what the developers returns if isRenderedAllowed() returns > false then it won't be rendered. > > johan > > > On 11/2/07, Igor Vaynberg <[EMAIL PROTECTED]> wrote: > > > > security is bypassed anyways because most component writers will > > forget to do the double check. so neither solution is good. > > > > -igor > > > > > > On 11/2/07, Johan Compagner <[EMAIL PROTECTED]> wrote: > > > > > > > > > > > > for visibility it is currently: isVisible()&&isRenderAllowed() which > > > > makes little sense to me because i have to deal with two concepts: > > > > visibility and rendering. from my point of view as a user i dont care > > > > to know about rendering, i just want to plop my components down and > > > > tweak their visibility. > > > > > > > > > so just introduce an extra final method on component that does just that > > > check. > > > > > > > > > when we first introduced this i argued to make isenabled() and > > > > isvisible() include the is*allowed() checks, but i didnt win that one > > > > back then...but thats another thread. > > > > > > > > > no i still think thats a bad idea, because then isEnabled and isVisible > > > must be both final i guess > > > because then with simple isVisible override by some developer the > > security > > > is by passed. > > > > > > johan > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
