Thanks for the explanation! I will modify the code according to the principles described. ;-)
see you soon, ruesp83 2013/12/16 Campbell Barton <[email protected]> > Yep, check is to validate settings to avoid having to maintain a valid > state within draw callback for eg which is now disallowed since it > caused too many problems. > Its also used to ensure ".blend" extension is used when saving. > > Don't think check() should be reporting back to the user, perhaps we > could print reports into the stdout so at least they are not lost. > > They could be displayed in the info header too. > > On Tue, Dec 17, 2013 at 4:51 AM, Brecht Van Lommel > <[email protected]> wrote: > > Ah, this seems to be a new callback. I think the current purpose with > > the image save operator is to validate the settings before the users > > actually does anything with the operator, it's to correct remembered > > properties. So for that kind of purpose there should be no message, it > > should just silently fix or change the property. But maybe I'm not > > understanding this correctly, Campbell added this. > > > > In the example you give, a message like "The offset can not be larger > > than the image" is not particularly useful anyway, if the offset is > > bigger than the image then that's not really a problem? If the > > integers are only valid in some range, ideally the button wouldn't > > allow you enter them in the first place as well. > > > > Brecht. > > > > On Mon, Dec 16, 2013 at 5:12 PM, Fabio Russo <[email protected]> wrote: > >> I refer to such a situation: http://www.pasteall.org/48064/c > >> > >> > >> 2013/12/16 Brecht Van Lommel <[email protected]> > >> > >>> What is the operator "check", can you be more precise and explain > >>> exactly where you are putting BKE_report call? > >>> > >>> > >>> On Mon, Dec 16, 2013 at 4:17 PM, Fabio Russo <[email protected]> > wrote: > >>> > Hello to all, > >>> > If we use inside the operator "check" the function BKE_report, it is > not > >>> > displayed in the UI. > >>> > This is normal behavior or is it a bug? > >>> > > >>> > Thank you, > >>> > ruesp83 > >>> > _______________________________________________ > >>> > Bf-committers mailing list > >>> > [email protected] > >>> > http://lists.blender.org/mailman/listinfo/bf-committers > >>> _______________________________________________ > >>> Bf-committers mailing list > >>> [email protected] > >>> http://lists.blender.org/mailman/listinfo/bf-committers > >>> > >> _______________________________________________ > >> Bf-committers mailing list > >> [email protected] > >> http://lists.blender.org/mailman/listinfo/bf-committers > > _______________________________________________ > > Bf-committers mailing list > > [email protected] > > http://lists.blender.org/mailman/listinfo/bf-committers > > > > -- > - Campbell > _______________________________________________ > Bf-committers mailing list > [email protected] > http://lists.blender.org/mailman/listinfo/bf-committers > _______________________________________________ Bf-committers mailing list [email protected] http://lists.blender.org/mailman/listinfo/bf-committers
