Marco Pivetta
e1e7485e58
Correcting the SimpleInsertPerformanceBench, which was running into duplicate record integrity constraint violations when run with >1 revs
2017-08-26 16:41:03 +02:00
Marco Pivetta
7a0385634f
Merge pull request #6651 from doctrine/fix/use-quote-model-set-when-available
...
Replacing direct `quote` model-set schema creation with actually existing model-set setup
2017-08-26 14:23:26 +02:00
Marco Pivetta
922c55f5dd
Replacing direct quote model-set schema creation with actually existing model-set setup
2017-08-26 14:15:05 +02:00
Marco Pivetta
61404e2d6d
Merge pull request #6640 from doctrine/fix/#6284-#6217-avoid-passing-l2-cache-information-internals-to-the-uow
...
#6217 #6284 when hydrating an entity with a composite primary key that is both an `EAGER` and a `LAZY` association and also cached, the `DefaultQueryCache` tries to pass L2 cache implementation detail objects to the `UnitOfWork`
2017-08-25 09:22:45 +02:00
Marco Pivetta
660f164568
#6284 #6217 s/$this->assert/self::assert as per @lcobucci's review
2017-08-25 09:21:12 +02:00
Marco Pivetta
acfc890dbd
Merge pull request #6644 from AlessandroMinoccheri/fix_documentation_typo
...
fixed typo into native-sql documentation
2017-08-24 11:29:17 +02:00
Alessandro Minoccheri
a708d4076e
fixed typo into native-sql documentation
2017-08-24 11:27:54 +02:00
Marco Pivetta
aac0204611
Merge pull request #6642 from radmax/patch-1
...
Fixed typo in DQL docs
2017-08-24 10:57:11 +02:00
radmax
ed047520c3
fixed typo
2017-08-24 10:55:29 +02:00
Marco Pivetta
caa008b61d
#6284 #6217 removing hacks around the ClassMetadata details - invalid fix that was actually fixing the symptom
2017-08-23 00:25:29 +02:00
Marco Pivetta
07b397f341
#6284 fixing actual issue, which comes from an extremely tricky L2 caching issue.
...
We are not hydrating some of the cached association data into entities due to keys missing in the cache association definition.
Since this is an extreme edge case that is just a mismatch between db and cache, a detailed explanation was provided in the fix snippet as well
2017-08-23 00:19:49 +02:00
Marco Pivetta
c9d1f852de
#6284 removing unused lazy entity
2017-08-22 21:41:04 +02:00
Marco Pivetta
c29a1e96b7
#6284 removing unused separate class
2017-08-22 21:40:42 +02:00
Marco Pivetta
a2f4053a81
#6284 renaming variables, classes and properties to fit tested behavior
2017-08-22 21:38:44 +02:00
Marco Pivetta
3f09e20955
#6284 correcting alignment, removing stuff that isn't strictly needed
2017-08-22 21:31:01 +02:00
Marco Pivetta
220dc79ebf
#6284 renaming entities to match the scope of this test
2017-08-22 21:29:27 +02:00
Marco Pivetta
6f6e88cfb6
#6284 adding assertions about equality of the loaded classes
2017-08-22 21:27:46 +02:00
Marco Pivetta
dac1ce4172
#6284 removing profile entity, since it is not needed to reproduce the issue
2017-08-22 21:21:53 +02:00
Marco Pivetta
ca39abcd71
#6284 reducing annotation mapping clutter
2017-08-22 21:17:14 +02:00
Marco Pivetta
4e0b76ce69
#6284 eager fetching is strictly required to verify the issue
2017-08-22 21:14:38 +02:00
Marco Pivetta
805ba041ef
#6284 reducing annotation mapping clutter
2017-08-22 21:12:45 +02:00
Marco Pivetta
3842ad8ea1
#6284 simplifying tests to a minimum, removing unused auto-generated id field
2017-08-22 21:11:03 +02:00
Marco Pivetta
c7281f6ade
#6284 simplifying tests to a minimum, removing unused auto-generated id field
2017-08-22 21:08:06 +02:00
Marco Pivetta
a8453dda89
#6284 removing the "WTF" part of the logic - an association cache entry should never ever reach metadata
2017-08-22 21:03:14 +02:00
Waleed Gadelkareem
309b286ed3
Add tests for #6217
2017-08-22 19:45:26 +02:00
Marco Pivetta
8ad3dfd3bd
Merge branch 'fix/#1521-DDC-2922-defer-checking-non-persisted-entities-through-multiple-differently-mapped-cascading-associations'
...
Close #1521
Close DDC-2922
2017-08-22 10:26:15 +02:00
Marco Pivetta
645cccf2db
#1521 DDC-2922 adapting test so the UnitOfWork stops bragging about previous inconsistent states
2017-08-22 10:24:46 +02:00
Marco Pivetta
2be32f249c
#1521 DDC-2922 adapting UoW and exception implementation to the new specification
2017-08-22 09:57:57 +02:00
Marco Pivetta
4a007c76f5
#1521 DDC-2922 spec for a new exception endpoint that produces a multi-new-non-cascaded-entities error message
2017-08-22 09:56:43 +02:00
Marco Pivetta
a754eae0f0
#1521 DDC-2922 verifying that persistence operations will resume normally after a crash due to invalid new values detected on associations - tweaked test to make it fail
2017-08-22 08:51:27 +02:00
Marco Pivetta
89fbb6a060
#1521 DDC-2922 verifying that persistence operations will resume normally after a crash due to invalid new values detected on associations
2017-08-22 08:49:42 +02:00
Marco Pivetta
2751c0fff2
#1521 DDC-2922 minor code cleanup - renaming internal private methods, variables, removing redundant if/else
2017-08-21 20:08:20 +02:00
Marco Pivetta
87e8bccb11
#1521 DDC-2922 removed redundant integration test that was replaced by a unit test
2017-08-21 19:58:52 +02:00
Marco Pivetta
f39614136f
#1521 DDC-2922 reproducing test scenarios within the UnitOfWorkTest
2017-08-21 19:58:16 +02:00
Marco Pivetta
a3208f8d08
#1521 DDC-2922 removed unrelated model usage from test - using minimal models only
2017-08-21 19:47:16 +02:00
Marco Pivetta
e21b29c264
#1521 DDC-2922 simplified test case to a minimum reproducible unit
2017-08-21 19:44:38 +02:00
Darien Hager
b456cffa2d
Move final cascade-persist-checking so that it covers not just normal flushes, but also flushes where specific entities are singled out.
2017-08-21 09:08:31 +02:00
Darien Hager
17b996da8c
Speculative fix: Defer any errors for missing cascade-persist until object graph has been better-explored
2017-08-21 09:08:31 +02:00
Darien Hager
92dc39bfb9
Add extra assertions to test to ensure that flush actually succeeded in saving all items
2017-08-21 09:08:31 +02:00
Darien Hager
997000352a
Add another test to demonstrate the bug doesn't appear if everything is flushed at once.
2017-08-21 09:08:31 +02:00
Darien Hager
6f8a80be79
Move failing unit test into ticket-specific case
2017-08-21 09:08:31 +02:00
Darien Hager
c4465abaa0
Initial failing test-case to demonstrate cascade-persist problem.
2017-08-21 09:08:12 +02:00
Marco Pivetta
ddccd42bb1
Merge pull request #6636 from Kwadz/annot-ref-update
...
Mention the nullable default value for @column
2017-08-20 08:17:41 +02:00
Richard
9eaf23a5e0
Mention the nullable default value for @column
...
Related to #6632
2017-08-20 05:57:00 +02:00
Marco Pivetta
a30d8d85ea
Merge branch 'fix/#6303-#6304-check-for-null-discriminator-before-hydrating-inheritance-rows'
...
Close #6303
Close #6304
ge aborts
2017-08-19 18:37:04 +02:00
Marco Pivetta
44f2e22f14
#6303 #6304 documenting why collisions in field name hydration in STI/JTI require additional information and checks in the hydration process
2017-08-19 18:30:33 +02:00
Marco Pivetta
4cbcdb761a
#6303 #6304 documenting that the discriminator map order is extremely important for this test
2017-08-19 18:25:44 +02:00
Marco Pivetta
b66643d52e
#6303 #6304 removing useless union type usage
2017-08-19 17:49:14 +02:00
Marco Pivetta
42d9162bd5
#6303 #6304 re-enabling disabled test code data
2017-08-19 17:47:12 +02:00
Marco Pivetta
8af68614fc
#6303 #6304 correcting type mapping. For simple_array, [] == null (which is bullshit), so we use array here
2017-08-19 17:45:26 +02:00
