Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
poll.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
21 May 2012 at 18:50 UTC
Updated:
29 Jul 2014 at 20:42 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
aspilicious commentedComment #2
Tor Arne Thune commentedShould this be PollJsAddChoiceTest?
Other than that this looks good.
Comment #3
robloachThe old one was class PollJSAddChoice, so PollJSAddChoiceTest.php seems like the right spot for it. RTBC! We could make a follow up to clean up the definition or name if we wanted.
Comment #4
Tor Arne Thune commentedSorry, I meant the JS acronym capitalization. Shouldn't it be Js in CamelCase?
Comment #5
robloachDoes just switching the case work? Lazy patch is lazy :-) .
Comment #6
Tor Arne Thune commentedLooks good! That's what I meant.
Comment #7
catchCommitted/pushed to 8.x.