Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
comment.module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
24 Nov 2009 at 00:34 UTC
Updated:
3 Jan 2014 at 00:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
plachSee #539110-61: TF #4: Translatable fields UI for a detail.
Comment #2
plachIMO this issue is only an exception: we need those tags primarly beacuse in the case of comments the item list is embedded into another's entity renderable content, so it's not that easy to replace it with a view (and core can't do that), I can't think of other item lists in core with the same problems.
Comment #3
sunerr... this code just adds context, but doesn't change any behavior and doesn't add any visible functionality. Can you explain again why we need this?
Comment #4
plachIn TF4 we need this to allow to filter comment lists per language, this way different translations may appear as different nodes.
Comment #5
sunAlrighty!
Comment #6
webchickI had asked plach on IRC whether it made sense to generalize this out a bit, since it seemed odd to add such metadata to comment queries and not others. He said that comments are really the only thing, in core at least, that's tricky like this because it's inherently tied to a node, and needs to get translation data from that.
Committed to HEAD!
Comment #7
plachyay!