[Bf-committers] BKE_report in operator "Check"

[Fabio Russo]

Thanks for the explanation!
I will modify the code according to the principles described. ;-)

see you soon,
ruesp83


2013/12/16 Campbell Barton <ideasman42 at gmail.com>

> 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
> <brechtvanlommel at pandora.be> 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 <ruesp83 at gmail.com> wrote:
> >> I refer to such a situation: http://www.pasteall.org/48064/c
> >>
> >>
> >> 2013/12/16 Brecht Van Lommel <brechtvanlommel at pandora.be>
> >>
> >>> 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 <ruesp83 at gmail.com>
> 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
> >>> > Bf-committers at blender.org
> >>> > http://lists.blender.org/mailman/listinfo/bf-committers
> >>> _______________________________________________
> >>> Bf-committers mailing list
> >>> Bf-committers at blender.org
> >>> http://lists.blender.org/mailman/listinfo/bf-committers
> >>>
> >> _______________________________________________
> >> Bf-committers mailing list
> >> Bf-committers at blender.org
> >> http://lists.blender.org/mailman/listinfo/bf-committers
> > _______________________________________________
> > Bf-committers mailing list
> > Bf-committers at blender.org
> > http://lists.blender.org/mailman/listinfo/bf-committers
>
>
>
> --
> - Campbell
> _______________________________________________
> Bf-committers mailing list
> Bf-committers at blender.org
> http://lists.blender.org/mailman/listinfo/bf-committers
>