EntityManager should not be used in code base. We have to use EntityTypeManager
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | poll_entity_manager_2870588_10-interdiff.txt | 3.44 KB | berdir |
| #10 | poll_entity_manager_2870588_10.patch | 5.5 KB | berdir |
| #7 | poll_entity_manager_2870588_7.patch | 6.98 KB | nkoporec |
| #4 | poll_entity_manager_2870588_4.patch | 5.55 KB | nkoporec |
Comments
Comment #2
Munavijayalakshmi commentedComment #3
dhansen commented@Munavijayalakshmi Looks like you mistakenly posted your patch for 2870586 here.
Comment #4
nkoporecCreated a patch for this issue.Please reviewed it.
Comment #5
icurk commentedDeprecated service was correctly replaced. I also noticed that dependency injection is almost never used. But for this a new issue should be created.
Comment #6
berdirThis part overlaps with another issue that properly injects this service. Lets remove it from that patch.
This should also be injected.
And this too.
Comment #7
nkoporecCreated new patch with fix issues that Berdir has raised.
Comment #8
lomasr commentedApplied the patch and it worked cleanly for me.
Comment #9
lomasr commentedComment #10
berdirDid more cleanup, one poll storage was actually unused, also missing docs and incorrect variable names.
When working on an existing patch, please always provide an interdiff.
Comment #12
berdirCommitted.