Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
theme system
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
7 Oct 2012 at 15:55 UTC
Updated:
29 Jul 2014 at 21:17 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
psynaptic commentedPatch attached.
Comment #2
andypostSo we need a test for this! Not sure it's critical
Comment #3
psynaptic commentedComment #4
fabianx commentedFound as part of twig engine conversion.
I am not sure this really needs a test and how to test this. Any ideas?
I think it is bad practice to use a class inside of a namespace and not add a use statement.
So I'd rather say this is a regression of the Attribute patch as calling new Attribute() from a different namespace can error out with fatal error.
In this case a new Attribute() added to template_preprocess_table lead to fatal error of:
Comment #5
fabianx commentedUnassign again due to x-post.
Comment #6
eric_a commentedFound another one. Setting status to needs review for the bot.
Comment #7
eric_a commentedSo not one test in core actually uses these iterators?
Comment #8
berdirAccording to the new guideline, global namespace objects should be referenced using \ArrayIterator, without explicit use statements.
Comment #10
rbayliss commentedHere are two patches that provide a test and bring these two files up to standards in light of #8. The first one (test only) should fail, the second one should pass.
Comment #11
rbayliss commentedWhoops. Set status.
Comment #12
tim.plunkettLooks good to me.
Comment #13
webchickAwesome, thanks for the fix and for the tests.
Committed and pushed to 8.x.