ie:
rid node_type mid language weight condition
3 news 23 es 0
4 news 25 fr 0

CommentFileSizeAuthor
#4 node_breadcrumb.png17.26 KBdvinegla
#1 node_breadcrumb2_i18n.patch26.26 KBdvinegla

Comments

dvinegla’s picture

dvinegla commented
Status: Active » Needs work
StatusFileSize
new node_breadcrumb2_i18n.patch26.26 KB

i18n patch after #253007: Clean up and code style

edhel’s picture

edhel commented

1) does this feature depend on i18n module?
2) I suppose that second "ALTER TABLE" is unnecessary in node_breadcrumb_update_1().
3) really you can use "php condition" for this, it's quite simple php-code: $node->language == 'LANG'

dvinegla’s picture

dvinegla commented

1) It works without any dependences, but if i18n is enabled, you can assign different breadcrumb to node translations.
2) Maybe. I'm not sure how hook_update works.
3) OK, with php condition works fine.

dvinegla’s picture

dvinegla commented
StatusFileSize
new node_breadcrumb.png17.26 KB

However, the first row looks like more usable than the second one.

Alice Heaton’s picture

Alice Heaton commented

Hi,

I can see this issue is about the 5.x version - in case someone else looks for this in the 6.x version this can be easilly done with adding a PHP condition of the type "$node->language == 'fr'"

Thanks,
Anselm

edhel’s picture

edhel commented
Assigned: Unassigned » edhel
Status: Needs work » Closed (won't fix)