Bug 18153 - NSFetchRequest.Predicate property setter should allow null
Summary: NSFetchRequest.Predicate property setter should allow null
Status: VERIFIED FIXED
Alias: None
Product: iOS
Classification: Xamarin
Component: Xamarin.iOS.dll ()
Version: master
Hardware: Macintosh Mac OS
: Normal normal
Target Milestone: 7.2.1
Assignee: Sebastien Pouliot
URL:
Depends on:
Blocks:
 
Reported: 2014-03-04 08:21 UTC by Nick Rymanov
Modified: 2014-03-20 05:50 UTC (History)
4 users (show)

Tags:
Is this bug a regression?: ---
Last known good build:

Notice (2018-05-24): bugzilla.xamarin.com is now in read-only mode.

Please join us on Visual Studio Developer Community and in the Xamarin and Mono organizations on GitHub to continue tracking issues. Bugzilla will remain available for reference in read-only mode. We will continue to work on open Bugzilla bugs, copy them to the new locations as needed for follow-up, and add the new items under Related Links.

Our sincere thanks to everyone who has contributed on this bug tracker over the years. Thanks also for your understanding as we make these adjustments and improvements for the future.


Please create a new report on Developer Community or GitHub with your current version information, steps to reproduce, and relevant error messages or log files if you are hitting an issue that looks similar to this resolved bug and you do not yet see a matching new report.

Related Links:
Status:
VERIFIED FIXED

Description Nick Rymanov 2014-03-04 08:21:45 UTC
NSFetchRequest.Predicate property setter should allow null
Comment 1 Sadik Ali 2014-03-04 09:11:50 UTC
I checked this issue and noticed that NSFetchRequest.Predicate property setter not allowed null
Refer screen cast: http://screencast.com/t/9hw3SGztcj

Checked With

All Mac
XS 4.2.3(build 59), 4.2.4 (build 23 )
X iOS 7.0.6.168
Comment 2 Sebastien Pouliot 2014-03-04 09:44:33 UTC
Documentation is unclear - but implies `nil` is valid. I'll test it to confirm then remove the extraneous check (if required).

I'll also audit other members of this type (to see if others should be accepting null values).
Comment 3 Sebastien Pouliot 2014-03-04 11:57:13 UTC
Note: The extra null check was also removed from other properties.

Fixed in master / 8bc9dd525438f68a2ac52bd2a164d6e94ab8c64c

QA: unit tests added in the same revision
Comment 4 Ram Chandra 2014-03-20 05:50:35 UTC
I have checked this issue on following builds:

Mac 10.7.5
Xamarin Studio : 4.2.4 (build 32)
Xamarin.iOS : 7.2.1.20 (Trial Edition)
Build Information
Release ID: 402040032
Git revision: a160c35dac9ab9fd32eeadaa171216316d5a5133
Build date: 2014-03-12 13:55:08-04
Xamarin addins: a779416ceabd54981ce812771a4061c942e8b872

I observe when I set the "Predicate" property of "NSFetchRequest class" it is not throwing the null reference exception and we are able to assign null to the "Predicate" property. I have checked the declaration of
"Predicate" property and  it is allowing null value. 

screencast:  http://www.screencast.com/t/4IkpaB3t

This issue is working fine. Hence, I am closing this issue.

Please let me know if I have to check anything else.