When editing templates, they should be able to be marked as translations of one another. Then, when creating an instance, a book is created for each translation, and the new nodes have the proper translation links set up so languages can easily be switched.

This isn't something I need for my current project, but I'm guessing most users of this module needing translation will want this feature. So, if you're interested, post a patch and I'll be glad to review it.

CommentFileSizeAuthor
#4 350325_template_translate_support.patch25.33 KBdeviantintegral
#3 350325_template_translate_support.patch25.32 KBdeviantintegral
#2 350325_template_translate_support.patch25.32 KBdeviantintegral
#1 350325_template_translate_support.patch25.32 KBdeviantintegral

Comments

deviantintegral’s picture

deviantintegral commented
Status: Postponed » Needs review
StatusFileSize
new 350325_template_translate_support.patch25.32 KB

Here is a patch which implements most of this feature. Most of it mirrors the i18n / translate UI and code, however we can't simply use those module's functions directly as they assume you're working with nodes.

Currently implemented:

  1. Migration of template node languages, if set, into a language column.
  2. API for loading all translations of a skeleton template with skeleton_template_load_translations()
  3. The ability to set templates as translations of each other, just like i18n.
  4. The ability to mark translations as needing to be translated. There is no UI yet for viewing all outdated translations.
  5. The ability to mark instantiated skeletons as translations of each other through the Syncronize interface. This is the roughest UI and will certainly be changed.

Unless I run into any significant issues, I will likely commit this patch and open followup issues for the remainder of the items.

deviantintegral’s picture

deviantintegral commented
StatusFileSize
new 350325_template_translate_support.patch25.32 KB

Here is an update which fixes a <> 0 with a = 0.

deviantintegral’s picture

deviantintegral commented
StatusFileSize
new 350325_template_translate_support.patch25.32 KB

Yet another typo of module_exists('translate') instead of module_exists('translation');

deviantintegral’s picture

deviantintegral commented
StatusFileSize
new 350325_template_translate_support.patch25.33 KB

Bah, lets upload the right file.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

Just to say, this is pretty freaking cool.

I may have to install just to test this....

deviantintegral’s picture

deviantintegral commented
Status: Needs review » Fixed

I've committed the patch from #4, and opened the following related issues. We've got this deployed on a live site now, and overall it seems to be working pretty well.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.