This repository was archived by the owner on Jun 16, 2024. It is now read-only.
Update to include till end of day in between dates query#588
Open
Skubakoob wants to merge 1 commit intojordimontana82:masterfrom
Open
Update to include till end of day in between dates query#588Skubakoob wants to merge 1 commit intojordimontana82:masterfrom
Skubakoob wants to merge 1 commit intojordimontana82:masterfrom
Conversation
|
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
Author
|
I have read the CLA Document and I hereby sign the CLA |
Author
|
Hmm seems the build is failing because of the last-week bug. I'll try it again tomorrow |
TarogStar
pushed a commit
to TarogStar/fake-xrm-easy
that referenced
this pull request
Nov 11, 2025
…ull handling This commit implements the top 3 priority quick wins identified from upstream repository analysis, resolving multiple long-standing issues reported by users. **Quick Win jordimontana82#1: Fix Date Range Operators** (Resolves jordimontana82#588, jordimontana82#587, jordimontana82#551, jordimontana82#543) - Modified XrmFakedContext.Queries.cs to include full end day (23:59:59.999) - Fixed: ThisMonth, LastMonth, NextMonth, ThisYear, LastYear, NextYear - Fixed: ThisWeek, LastWeek, NextWeek, InFiscalYear - Fixed: Between operator for date ranges - Added DateRangeOperatorTests.cs with 9 comprehensive tests **Quick Win jordimontana82#2: EntityReference.Name Population** (Resolves jordimontana82#555) - Modified RetrieveRequestExecutor.cs to populate EntityReference.Name - Modified RetrieveMultipleRequestExecutor.cs to populate EntityReference.Name - Automatically populates Name from referenced entity's primary name attribute - Added EntityReferenceNameTests.cs with 9 comprehensive tests **Quick Win jordimontana82#3: Null Reference Exception Fixes** (Resolves jordimontana82#608, jordimontana82#607) - Added defensive null checks in TranslateConditionExpressionLike - Added defensive null checks in TranslateConditionExpressionContains - Added defensive null checks in TranslateConditionExpressionEndsWith - Added defensive null checks in TranslateConditionExpressionIn - Prevents crashes when null values present in query conditions - Added NullReferenceHandlingTests.cs with 15 comprehensive tests **Test Coverage:** - 33 new tests total across 3 test files - All tests verify both functionality and edge cases - Tests cover QueryExpression and FetchXML scenarios **Impact:** - Resolves 8+ upstream issues reported by community - Improves compatibility with real Dataverse behavior - Prevents common crashes in query engine - Enhances developer experience with auto-populated EntityReference names All changes are backward compatible and follow existing code patterns.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.