Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
views.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
24 May 2013 at 15:31 UTC
Updated:
29 Jul 2014 at 22:24 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
jastraat commentedComment #2
jastraat commentedComment #3
aspilicious commentedThese should all be protected. ==> protected function valueForm()
Comment #4
zschmid commentedupdated patch
Comment #5
zschmid commentedComment #6
aspilicious commentedThis shouldn't be here :)
protected
Comment #7
zschmid commentedCrap. Sorry about those extra files, must have forgotten to switch branches before this. Anywayss, i think it needed to be rerolled anyways after the core commits yesterday/today. Should be okay now.
Comment #8
zschmid commentedComment #10
E.Kurko commentedI'll check. CodeSprintUA
Comment #11
E.Kurko commentedAssigned to me
Comment #12
vito_a commentedRe-rolling the #7.
Comment #13
vito_a commentedComment #14
vito_a commentedComment #15
dawehnerPerfect
Comment #16
webchickCommitted and pushed to 8.x. Thanks!
Moving to the Views queue for the change notice.
Comment #17
xjmWe'll create a single change notice in #1856630: [Change notice] [META] Rename Views methods to core standards.