Download & Extend
Skeleton outlines » Issues

Allow templates to be linked together as translations of each other

Posted by deviantintegral on December 23, 2008 at 12:31am
Project:Skeleton outlines
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

Login or register to post comments

Comments

#1

Posted by deviantintegral on January 12, 2010 at 5:46pm
Status:postponed» needs review

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.

AttachmentSize
350325_template_translate_support.patch 25.32 KB

#2

Posted by deviantintegral on January 12, 2010 at 8:18pm

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

AttachmentSize
350325_template_translate_support.patch 25.32 KB

#3

Posted by deviantintegral on January 12, 2010 at 8:22pm

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

AttachmentSize
350325_template_translate_support.patch 25.32 KB

#4

Posted by deviantintegral on January 12, 2010 at 8:25pm

Bah, lets upload the right file.

AttachmentSize
350325_template_translate_support.patch 25.33 KB

#5

Posted by agentrickard on January 12, 2010 at 10:45pm

Just to say, this is pretty freaking cool.

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

#6

Posted by deviantintegral on January 15, 2010 at 2:44am
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.

#7

Posted by System Message on January 29, 2010 at 2:50am
Status:fixed» closed (fixed)

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