<div dir="ltr">As I see it will be fixed in version 7.4<br>Will it have an update for the current version?<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El lun, 28 jun 2021 a las 4:59, Uma S (<<a href="mailto:uma.s@vtiger.com">uma.s@vtiger.com</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi Henry,<div><br></div><div>This issue has been fixed, will be packed in the upcoming release - <a href="https://code.vtiger.com/vtiger/vtigercrm/commit/bc9f25988ddfff030e0a651ea1c9a6682bde4e10" target="_blank">Ref1</a>, <a href="https://code.vtiger.com/vtiger/vtigercrm/commit/b00bcb636dcbbfde89a351a3525968121c927754" target="_blank">Ref2</a></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 11, 2021 at 1:36 AM Rubén A. Estrada Orozco <<a href="mailto:rulotec1@gmail.com" target="_blank">rulotec1@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">But the question is, who is gonna be the hero who submits the fix? ;)<div><br clear="all"><div><div dir="ltr"><div dir="ltr">Saludos<div><br></div><div>Rubén</div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 10, 2021 at 2:14 PM Henry Cumbicus Rivera <<a href="mailto:hcumbicusr@gmail.com" target="_blank">hcumbicusr@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Well, in the environment where I had the problem, I already did the correction for || by && and now it works fine.<br><div><br></div><div>It would be nice if they add it to the version 7.3 patches and also to the next version.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">El jue, 10 jun 2021 a las 13:55, Alan Lord (<<a href="mailto:alanslists@gmail.com" target="_blank">alanslists@gmail.com</a>>) escribió:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I think just replacing the || with && is sufficient.<br>
<br>
Al<br>
<br>
<br>
On 10/06/2021 17:18, Rubén A. Estrada Orozco wrote:<br>
> Alan, I also had a hard time trying to figure it out. Part of the job of <br>
> a good developer is to create expressive code (not just code that works, <br>
> which is half the job at most, but code that can be easily understood <br>
> and maintained).<br>
> <br>
> With that goal in mind and using De Morgan as you propose, I would suggest:<br>
> <br>
> $viewBelongsToCurrentUser = ($customViewOwner == $currentUser->getId());<br>
> <br>
> if (!($currentUser->isAdminUser() || $viewBelongsToCurrentUser)) {<br>
>      throw new AppException("PERMISSION_DENIED");<br>
> }<br>
> <br>
> I think the above is more "brain-friendly".<br>
> <br>
> Saludos<br>
> <br>
> Rubén<br>
> <br>
> <br>
> On Wed, Jun 9, 2021 at 4:44 AM Sukhdev Mohan <<a href="mailto:s.mohan@myti.it" target="_blank">s.mohan@myti.it</a> <br>
> <mailto:<a href="mailto:s.mohan@myti.it" target="_blank">s.mohan@myti.it</a>>> wrote:<br>
> <br>
>     Also to be remembered or statements are shortcutted sonic the first<br>
>     statement is true it'll execute the block. While in an AND it's<br>
>     shortcutted if the first statement is false.<br>
> <br>
> <br>
>     Il mer 9 giu 2021, 09:38 Alan Lord <<a href="mailto:alanslists@gmail.com" target="_blank">alanslists@gmail.com</a><br>
>     <mailto:<a href="mailto:alanslists@gmail.com" target="_blank">alanslists@gmail.com</a>>> ha scritto:<br>
> <br>
>         Comparing two not tests often throws me too and it turns out<br>
>         it's even a<br>
>         "thing".<br>
> <br>
>         <a href="https://en.wikipedia.org/wiki/De_Morgan%27s_laws" rel="noreferrer" target="_blank">https://en.wikipedia.org/wiki/De_Morgan%27s_laws</a><br>
>         <<a href="https://en.wikipedia.org/wiki/De_Morgan%27s_laws" rel="noreferrer" target="_blank">https://en.wikipedia.org/wiki/De_Morgan%27s_laws</a>><br>
> <br>
>          >     not (A or B) = not A and not B<br>
>          >     not (A and B) = not A or not B.<br>
> <br>
>         :-D<br>
> <br>
>         On 09/06/2021 08:25, Alan Lord wrote:<br>
>          > Henry, good catch! It would be good to create a new ticket<br>
>         explaining<br>
>          > this bug and to create a Merge Request with the fix. If you<br>
>         are not<br>
>          > able, let this list know and I am sure one of us can do it on<br>
>         your behalf.<br>
> <br>
>         _______________________________________________<br>
>         <a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a> <<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a>><br>
> <br>
>     _______________________________________________<br>
>     <a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a> <<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a>><br>
> <br>
> <br>
> _______________________________________________<br>
> <a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a><br>
> <br>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><br><div>---------------------------------------------------------------</div><div>Henry C.</div><div>Tel.: 956727976</div></div></div></div></div></div></div></div></div></div></div></div></div>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote></div>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr"><div dir="ltr">With<br>Best Regards<br>Uma.S<br><div>Vtiger Team</div></div></div>
_______________________________________________<br>
<a href="http://www.vtiger.com/" rel="noreferrer" target="_blank">http://www.vtiger.com/</a></blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><br><div>---------------------------------------------------------------</div><div>Henry C.</div><div>Tel.: 956727976</div></div></div></div></div></div></div></div></div></div></div></div></div>