I have created a Drupal 7 port of this module inspired by the current code in Drupal 8 which has not been backported yet.
Please add to this project.
I am willing to support/co-maintain if needed.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | 1795638-d7-port.patch | 8.68 KB | c31ck |
| #3 | translation_own_nodes-7.x-1.tar_.gz | 7.27 KB | jvizcarrondo |
| #2 | translation_own_nodes.zip | 9.78 KB | rv0 |
| #1 | translation_own_nodes.zip | 9.79 KB | rv0 |
| translation_own_nodes.zip | 9.68 KB | rv0 |
Comments
Comment #1
rv0 commentedForgot a small detail, needed to remove $langcode from node_access
Comment #2
rv0 commentedok more bug fix, took a bit too much of D8 stuff in there, dunno how i overlooked it ;)
Comment #3
jvizcarrondo commentedHi Wim Vanheste.
Much appreciated your help in migrating Translation own nodes to drupal 7.x, check your previous post and had problems with $node->langcode in _translation_own_nodes_tab_access function, I made a rapid development basing on the module code that generated this "Content translation" (translation in drupal core). I tested it and it seems to work, I put it in this post so it can be tested in the future and place it as a version of the module.
As I said before I think your work is greatly appreciated by the users of this module, I have noticed some additional hook that you use in the your code (hook_node_access), I do not understand well to do but I will not have problems in add this feature in the new version if fix a access problem or improvement in some performance module.
grateful
Juan
Comment #4
rv0 commentedyeah the langcode thing is what i fixed in #2 i think.
the node access code is taken from D8
i think it is for doing translations with other means than the standard /translate tab. (with a translation helper module or similar)
Comment #5
rv0 commentedPlease commit?
Comment #6
rv0 commentedI found another typo
Line 49, "translate all content" should be "translate content"
Otherwise the normal translation permission wont work.
Comment #7
c31ck commentedI've created a patch against 6.x-1.x based on #2 that:
- Fixes the issue mentioned in #6.
- Fixes some small coding standards violations.
- Cleans up trailing whitespace and missing newlines.
Comment #8
taitai commentedThe code in #3 will not work
Line 27: return TRUE will disable the whole point of this module. Needs to be commented out.
Comment #9
lklimek commentedI've applied #7 and it seems to work for me (it just lacks proper core version in .info file).
Please create 7.x branch with this :).
Comment #10
toaijala commentedI would like to get a 7.x version of this instead of hacking aswell..
Comment #11
rv0 commentedI forgot I had gained commit access to this project. Committed. Give it some time to appear on the page.
Comment #12
toaijala commentedI installed the dev package, it works great for me!