Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
documentation
Priority:
Minor
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
30 Jul 2012 at 09:59 UTC
Updated:
29 Jul 2014 at 20:55 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dsdeiz commentedChanging "Component". And "Category"? Sorry, I'm not sure if I chose the right category.
Comment #2
jhodgdonGood catch! Yes, the correct hook is hook_entity_query_alter().
Comment #3
dsdeiz commentedAttached.
Comment #4
jhodgdonThanks! I'll get that committed.
(By the way, I'd recommend not wasting time making D7 patches until/unless we need them. See
http://drupal.org/node/1319154#multiple-versions
for why.)
Comment #5
jhodgdonforgot status change. :)
Comment #6
dsdeiz commentedAh yeah. My bad. Won't happen again. The folks at #drupal-contribute also advised me this when I asked there.
Comment #7
chx commentedThe Drupal community never ceases to astonish me. You guys caught this? Amazing.
Comment #8
jhodgdonIt's not a *problem* that you made an extra patch, just often unnecessary work (and time you could be spending making another patch). In this case, the D7 patch was useful, because the file was a completely different name for D7. Normally, we would have wanted a clean patch to be uploaded so it could be run through the tests, but since it's a one-line docs change, I let it go this time. :)
Anyway, thanks again for contributing to Drupal! I've committed the above patches to D8 and D7.