| Comment | File | Size | Author |
|---|---|---|---|
| #11 | deploy-translations-first.patch | 1.46 KB | longwave |
| #10 | deploy-translations-first.patch | 1.47 KB | longwave |
| #3 | 642384_translation_support.patch | 2.7 KB | gdd |
| #2 | 642384_translation_support.patch | 3.09 KB | gdd |
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | deploy-translations-first.patch | 1.46 KB | longwave |
| #10 | deploy-translations-first.patch | 1.47 KB | longwave |
| #3 | 642384_translation_support.patch | 2.7 KB | gdd |
| #2 | 642384_translation_support.patch | 3.09 KB | gdd |
Comments
Comment #1
levacjeep commentedLet me know if you need a hand with this, I wouldn't mind contributing some code as I am working on a project that requires this functionality.
Cheers,
Jean-Pierre Levac
Comment #2
gddA long time coming but here is a patch to try
Comment #3
gddSorry that had some debug crap in it, here's a new patch
Comment #4
dixon_Is this fixed in HEAD? All code seems to be there except this part from the latest patch:
Comment #5
gddHm, both those hunks appear to have been added already by accident at this commit
http://drupalcode.org/viewvc/drupal/contributions/modules/deploy/modules...
I need to resolve that against the code I'm currently running on the client site I wrote this for. I could swear the second hunk went in deploy.module and not node_deploy.module. I'll look into this.
Comment #6
csavio commentedSubscribing. Did all of this patch get implemented in 1.1.2.3?
Comment #7
gddYup looks that way. Marking this fixed.
Comment #9
longwaveReopening this as I think I have discovered a bug. If you deploy a node that has translations, you must ensure that the source node for the translation exists on the destination before the translations get pushed over.
This is triggered if you are pushing new content in multiple languages, including the source translation node; you currently can't guarantee the order that the nodes will be created in, and if the translations end up being pushed before the source translation node, the translations will not be linked correctly.
Comment #10
longwaveThe attached patch tries to ensure the deployment order is correct when deploying a new node and any translations as part of the same plan.
I am not sure the plan item weight system was meant to be used in this way - there is no API call for what the patch does, but this approach seems to work so perhaps one should be added.
Comment #11
longwaveTypo in the SQL statements in that patch. Use this one instead.
Comment #12
dixon_I will take a closer look into this before rolling the beta. Adding tag for that.
Comment #13
dixon_Actually, I'm closing this one as a duplicate of #766806: Weight between items will break deployment if dependencies are added by hand in wrong order where we should try to solve this in a more general manner.
Comment #14
RichaDaffodil commentedI am also facing this issue. When i am deploying nodes with translation then on destination server tnid is saved to 0. I am using deploy module version 6.x-1.0-alpha2. Does it have all commits that are mentioned?