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.
When submitting a live job for a quote, neither Standard, Pro, nor Ultra show a quote based on the wordcount, although wordcount seems to be working correctly.
Comment | File | Size | Author |
---|---|---|---|
#11 | tmgmt_mygengo.module.patch | 554 bytes | devlada |
#4 | sandbox_response.jpeg | 189.11 KB | devlada |
Screen Shot 2012-06-07 at 5.08.43 PM.png | 39.59 KB | Gastonia |
Comments
Comment #1
BerdirThanks for testing and reporting useful bug reports :)
The MyGengo translator hasn't been touched much since this project was started with the initial sprint a few month back. I personally have never really tried it out so far.
It is very likely that there are bugs in our implementation and it's also possible that their API has changed and that we're no longer compatible.
Keep that in mind when testing, I strongly suggest to stick with the test environment for now, if you're not doing so already.
Comment #2
BerdirPS: I'm available in #drupal-i18n/#drupal-contribute IRC channels if you want to discuss something.
Comment #3
devlada CreditAttribution: devlada commentedComment #4
devlada CreditAttribution: devlada commentedAs you can see in attached file, sandbox return 0 for credits field for both jobs (title and body). There are two mygengo translators (mygengo_autoaccept and mygengo) and both behave the same. Currently I can not test live API to could compare.
Comment #5
miro_dietikerIn that case we should put a message there to state we're in sandbox mode...
Additionally we can pass you credentials to test mygengo live. (you even don't need to put a real job there... requesting the price is enough.)
Comment #6
devlada CreditAttribution: devlada commentedComment #7
BerdirI'm pretty sure that the original report is from the live system, so I don't think it's just the test system that is affected.
But yes, you can just create an account on the live system and try to get a quote.
Also have a look at the actual response from MyGengo. It's possible that due to a bug on our side or an API change ( the current code explicitly has some live/sandbox-specific code, it's quite possible that live is now running with the same API as the sandbox).
If it's reported as 0 in there both on live and on sandbox, we should report it to MyGengo.
Comment #8
devlada CreditAttribution: devlada commentedAfter better check, we have next problems here:
- when mashine quality is set, for some languages sandbox returns 0 for credits field (previously reported), but for some (Italian) returns error code 1551 with message "language service not supported".
- in case when sandbox return error, Word count is reset to 0.
- when we change options for To and Quality fields, Needed Credits not refreshing. Callback methods should be improved to properly refresh the data.
- In case of errors it is necessary to show a message about it and to prevent Save job action.
Comment #9
devlada CreditAttribution: devlada commentedI checked the live API, the same behavior as the sandbox, I did not notice any difference in the response and behavior.
Comment #10
BerdirOk, debugged it a bit myself.
- Spanish and Italian seem to be broken, please open a separate issue for this. Our calls look ok to me, not sure where the problem is.
- The API and the plugin is working, the only thing that is broken the our ajax form handling.
in the quote ajax callback, there is this line:
return $form['translator_wrapper']['options']['settings']['quote'];
The options key is too much, that doesn't exist anymore. Remove ['options'] and it works.
=> The language switcher probably has a similar problem, that one doesn't seem to be return anything from the ajax callback. Also a separate issue as that is not specific to mygengo.
Comment #11
devlada CreditAttribution: devlada commentedPatch for quote callback.
Comment #12
devlada CreditAttribution: devlada commentedComment #13
BerdirRemember to set issues to needs review when uploading patches.
Comment #14
BerdirCommited.
Comment #15
BerdirMoving this issue over to the new Mygengo Project.