[Bf-committers] BKE_report in operator "Check"

[Campbell Barton]

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