<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2013/11/19 Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class=""><div class="h5">Piotr Przybylski a écrit :<br>
> 2013/11/19 Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>><br>
><br>
>> Piotr Przybylski a écrit :<br>
>>> 2013/11/19 Marc Delisle <<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>><br>
>>><br>
>>>> Le 2013-11-18 19:37, Mohamed Ashraf a écrit :<br>
>>>>> On Monday, November 18, 2013 at 7:06 PM, Marc Delisle wrote:<br>
>>>>>> Hi Mohamed,<br>
>>>>>><br>
>>>>>> Why do we need two configuration directives? I guess that instead of<br>
>>>>>> verifying whether ErrorReporting is true, we could just verify whether<br>
>>>>>> SendErrorReports is different than 'never' to load the js files.<br>
>>>>>><br>
>>>>>> If you confirm my suggestion, I'll make the changes.<br>
>>>>>><br>
>>>>> One was actually meant for the hoster of the phpmyadmin installation<br>
>> and<br>
>>>>> one for the user. If you donot want the phpmyadmin deployer to have<br>
>>>>> control over it you can remove it.<br>
>>>>><br>
>>>>> I just thought this is a new system and if a fatal bug is not caught<br>
>>>>> before a release we need to have an easy way for a deployer using that<br>
>>>>> release to disable it.<br>
>>>>><br>
>>>>> Or if for whatever reason the data is secretive enough that he diesnt<br>
>>>>> want it on our publc servers if he doesnt trust it is anonymous enough.<br>
>>>> Thanks, I accept your explanation and I'll just add some documentation.<br>
>>>><br>
>>>><br>
>>> Hosters can disable ErrorReporting and add it to UserprefsDisallow:<br>
>>> $cfg['UserprefsDisallow'] = array('UserprefsDisallow');<br>
>>><br>
>>> That way users won't be able to override their settings.<br>
>> Piotr,<br>
>> ErrorReporting is not part of user preferences, but SendErrorReports is.<br>
>><br>
>><br>
> Yes, you are right. I just gave it as an example that it can be done with<br>
> just one switch.<br>
<br>
</div></div>I see what you mean. So we don't need the ErrorReporting directive. To<br>
block the feature completely, the deployer would do<br>
<br>
$cfg['SendErrorReports'] = 'never';<br>
$cfg['UserprefsDisallow'] = array('SendErrorReports');<br>
<div class=""><div class="h5"><br></div></div></blockquote><div><br></div><div>Exactly, that's what this function is designed to do. We just need to add SendErrorReports to setup script in place of ErrorReporting. Then setup script will display a checkbox that allows for adding it to UserprefsDisallow in generated config (right aligned checkbox at [1]).<br>
<br>[1] <a href="http://demo.phpmyadmin.net/master/setup/index.php?page=form&formset=Features#tab_Other_core_settings">http://demo.phpmyadmin.net/master/setup/index.php?page=form&formset=Features#tab_Other_core_settings</a><br>
<br>-- <br></div><div>Regards,<br></div><div>Piotr Przybylski<br></div></div></div></div>