Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This was reported by a customer. I went step-by-step with him over the phone.
The issue is that when removing the "sandbox" checkbox (and putting the production API keys), the module does not allow to translate the node with OHT.
How we reproduced this problem:
1. Enter "sandbox API keys" on the module settings
2. Select "Sandbox" checkbox
3. Submit a node for translation -> the node is submitted and translation is returned. Everything OK.
4. Uncheck "sandbox" checkbox and enter production API keys
5. Unable to submit any node for translation.
Comment | File | Size | Author |
---|---|---|---|
#8 | fix-supported-langs-detection-2107267-8.patch | 945 bytes | oyagev |
#4 | fix-supported-langs-detection-2107267-4.patch | 939 bytes | oyagev |
#2 | Settings Screen | 93.18 KB | oyagev |
#2 | Node Translation | 139.95 KB | oyagev |
Comments
Comment #1
miro_dietikerWe need language setups (keys) and language mapping definitions, and especially the node source language and target language as such as possibly other checkout settings that are in place.
Comment #2
oyagev CreditAttribution: oyagev commentedHi
I took some screenshots, hope this helps...
Comment #3
oyagev CreditAttribution: oyagev commentedComment #4
oyagev CreditAttribution: oyagev commentedFound a problem in "canTranslate" function
Please see patch
Comment #5
oyagev CreditAttribution: oyagev commentedComment #6
BerdirComment #7
BerdirFix looks correct, but you accidently changed the indentation to 4 spaces instead of 2 for those lines.
If you have time to upload a patch with that fixed, I'll commit it asap. Otherwise I'll get to it in the next few days.
Comment #8
oyagev CreditAttribution: oyagev commentedSure
Although time was of the essence, I already committed and customer already tested and approved.
Here's a patch that fixes the indentation.
Comment #9
BerdirAh right, I forgot that you have commit access too.
Looks like your e-mail isn't set up correctly, though. See https://drupal.org/node/1612398/commits, your commit isn't linked to your username and therefore also doesn't show up in the recent commits block on the project. Make sure that the e-mail address that is configured in your git client matches one of the e-mails that you've set up on drupal.org (you can add multiple ones in your profile).
Comment #10
oyagev CreditAttribution: oyagev commentedYes I noticed that too.
Already fixed.
Should I commit the new fix (indentation) ?
Comment #11
BerdirYes, go ahead :)
Comment #12
oyagev CreditAttribution: oyagev commentedpushed