Re: [PATCH] fix for bug in SRecordGeneric
Noel Grandin <[email protected]> Mon, 04 Jun 2012 12:30:18 +0200
| Newsgroups | gmane.comp.java.orm.simpleorm |
|---|---|
| Message-ID | <[email protected]> |
--------------070107000506020700010306
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 8bit
No, the problem is that some tools are using
new Boolean(true)
which results in the method returning false.
On 2012-06-04 11:36, Franck Routier wrote:
> Thanks Noel for the patch.
> Do you have more information about the problem this corrects ? I guess
> it must a NPE when the map contains a null value for the field, but I
> would just like to check it is not hidding another bug.
>
> Franck
>
>
>
> Le 04/06/2012 11:23, Noel Grandin a écrit :
>> [Attachment(s) <#TopText> from Noel Grandin included below]
>>
>> Hi
>>
>> There is a bug in SRecordGeneric.getBoolean that manifests in some odd
>> situations.
>>
>> Patch attached.
>>
>> Regards, Noel Grandin
>>
>> Disclaimer: http://www.peralex.com/disclaimer.html
>>
>>
>
Disclaimer: http://www.peralex.com/disclaimer.html
--------------070107000506020700010306
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
<head>
<style type="text/css">
<!--
/* start of attachment style */
.ygrp-photo-title{
clear: both;
font-size: smaller;
height: 15px;
overflow: hidden;
text-align: center;
width: 75px;
}
div.ygrp-photo{
background-position: center;
background-repeat: no-repeat;
background-color: white;
border: 1px solid black;
height: 62px;
width: 62px;
}
div.photo-title
a,
div.photo-title a:active,
div.photo-title a:hover,
div.photo-title a:visited {
text-decoration: none;
}
div.attach-table div.attach-row {
clear: both;
}
div.attach-table div.attach-row div {
float: left;
/* margin: 2px;*/
}
p {
clear: both;
padding: 15px 0 3px 0;
overflow: hidden;
}
div.ygrp-file {
width: 30px;
valign: middle;
}
div.attach-table div.attach-row div div a {
text-decoration: none;
}
div.attach-table div.attach-row div div span {
font-weight: normal;
}
div.ygrp-file-title {
font-weight: bold;
}
/* end of attachment style */
-->
</style>
</head>
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
No, the problem is that some tools are using <br>
new Boolean(true)<br>
which results in the method returning false.<br>
<br>
On 2012-06-04 11:36, Franck Routier wrote:
<blockquote cite="mid:[email protected]" type="cite">
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
Thanks Noel for the patch.<br>
Do you have more information about the problem this corrects ? I
guess it must a NPE when the map contains a null value for the
field, but I would just like to check it is not hidding another
bug.<br>
<br>
Franck<br>
<br>
<br>
<br>
Le 04/06/2012 11:23, Noel Grandin a écrit :
<blockquote cite="mid:[email protected]" type="cite"> <span
style="display:none"> </span>
<div id="ygrp-text">
<p>Hi<br>
<br>
There is a bug in SRecordGeneric.getBoolean that
manifests in some odd <br>
situations.<br>
<br>
Patch attached.<br>
<br>
Regards, Noel Grandin<br>
<br>
Disclaimer: <a moz-do-not-send="true"
href="http://www.peralex.com/disclaimer.html">http://www.peralex.com/disclaimer.html</a><br>
<br>
</p>
</div>
<!-- end group email --> </blockquote>
<br>
</blockquote>
<br><br><br><hr><font size="-2" color=808080>Disclaimer: <a href="http://www.peralex.com/disclaimer.html">http://www.peralex.com/disclaimer.html</a><br><br>
<!-- |**|begin egp html banner|**| -->
<br>
<br>
<!-- |**|end egp html banner|**| -->
<div width="1" style="color: white; clear: both;"/>__._,_.___</div>
<!-- Start Recommendations -->
<!-- End Recommendations -->
<!-- |**|begin egp html banner|**| -->
<img src="http://geo.yahoo.com/serv?s=97476590/grpId=7360226/grpspId=1705006905/msgId=2061/stime=1338805881" width="1" height="1"> <br>
<!-- |**|end egp html banner|**| -->
<!-- |**|begin egp html banner|**| -->
<br>
<div style="font-family: verdana; font-size: 77%; border-top: 1px solid #666; padding: 5px 0;" >
Your email settings: Individual Email|Traditional <br>
<a href="http://groups.yahoo.com/group/SimpleORM/join;_ylc=X3oDMTJmaTlyZHY5BF9TAzk3NDc2NTkwBGdycElkAzczNjAyMjYEZ3Jwc3BJZAMxNzA1MDA2OTA1BHNlYwNmdHIEc2xrA3N0bmdzBHN0aW1lAzEzMzg4MDU4ODE-">Change settings via the Web</a> (Yahoo! ID required) <br>
Change settings via email: <a href="mailto:[email protected]?subject=Email Delivery: Digest">Switch delivery to Daily Digest</a> | <a href = "mailto:[email protected]?subject=Change Delivery Format: Fully Featured">Switch to Fully Featured</a> <br>
<a href="http://groups.yahoo.com/group/SimpleORM;_ylc=X3oDMTJkYTFxYnBvBF9TAzk3NDc2NTkwBGdycElkAzczNjAyMjYEZ3Jwc3BJZAMxNzA1MDA2OTA1BHNlYwNmdHIEc2xrA2hwZgRzdGltZQMxMzM4ODA1ODgx">
Visit Your Group
</a> |
<a href="http://docs.yahoo.com/info/terms/">
Yahoo! Groups Terms of Use
</a> |
<a href="mailto:[email protected]?subject=Unsubscribe">
Unsubscribe
</a>
<br>
</div>
<br>
<!-- |**|end egp html banner|**| -->
<div style="color: white; clear: both;"/>__,_._,___</div>
</body>
</html>
--------------070107000506020700010306--