[dev] [commits] Horde branch master updated. e82250e4a2d28e0464ad26f3d6affb5ae51d80b2

[Michael J Rubinsky]

Quoting Michael J Rubinsky <mrubinsk at horde.org>:

> Quoting Michael J Rubinsky <mrubinsk at horde.org>:
>
>> Quoting Michael J Rubinsky <mrubinsk at horde.org>:
>>
>>> Quoting Jan Schneider <jan at horde.org>:
>>>
>>>>> commit 6697c620ead20833d8ca077f64e4cc26ed1ad8d8
>>>>> Author: Michael J Rubinsky <mrubinsk at horde.org>
>>>>> Date:   Wed Jun 24 12:31:18 2015 -0400
>>>>>
>>>>> Allow a string value of '0'.
>>>>>
>>>>> Part of fix for Bug: 14025
>>>>>
>>>>> framework/Perms/lib/Horde/Perms/Permission.php |    5 +++--
>>>>> 1 files changed, 3 insertions(+), 2 deletions(-)
>>>>>
>>>>> http://github.com/horde/horde/commit/6697c620ead20833d8ca077f64e4cc26ed1ad8d8
>>>>
>>>> How about checking for strlen() instead?
>>>
>>> I tried that, even when using strval() first, and it was still  
>>> taking it as an "empty" value.
>>
>> Though a quick test script shows that this should not be the case.  
>> I'll see if there was something else going on.
>
> Yeah, I think I tested that before changing the key values to be  
> strings in Horde :/  Updated in Git.

This also takes an integer value of 0 and returns a length of 1, so  
this technically breaks previous behavior of allowing an integer value  
of 0 "clear" the permission. Not sure if that falls under the  
"previous behavior is broken, so it's ok" school of breaking BC or  
not....

-- 
mike
The Horde Project
http://www.horde.org
https://www.facebook.com/hordeproject
https://www.twitter.com/hordeproject
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 5869 bytes
Desc: S/MIME Signature
URL: <http://lists.horde.org/archives/dev/attachments/20150624/26018d31/attachment.bin>