Changes between Initial Version and Version 1 of Ticket #42069, comment 10
- Timestamp:
- 06/26/2018 03:46:17 PM (7 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #42069, comment 10
initial v1 1 1 I cannot replicate this issue unfortunately. The tests you wrote all passed for me without actually using the changes in the code. @dcavins @MattGeri @adamsilverstein Can you provide some more background on your environment? I wonder if it's some PHP version-related detail or similar. 2 2 3 Regarding the patch, I just uploaded [attachment:42069.diff] with optimized tests using a data provider, which reduces the necessary code and at the same time tests a lot more combinations. There's one test for checking updating a value with exactly the same value, and another one for checking updating a value with another value that essentially means the same (like 0/1 or false/true or ''/'1'for booleans).3 Regarding the patch, I just uploaded [attachment:42069.diff] with optimized tests using a data provider, which reduces the necessary code and at the same time tests a lot more combinations. There's one test for checking updating a value with exactly the same value, and another one for checking updating a value with another value that essentially means the same (like 0/1 or false/true for booleans). 4 4 My new patch only includes the tests as they passed without the code change. I'm not at all saying that change is wrong, it just did not fix a problem for me, so I excluded it ''for now''. Of course, as we determine where exactly this problem is coming from, we can add the fix back as needed. Please use the new patch going forward and iterate from there.