Closed (fixed)
Project:
Poll
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
18 Apr 2017 at 07:37 UTC
Updated:
15 Jun 2018 at 21:04 UTC
Jump to comment: Most recent, Most recent file
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.