Problem/Motivation

To reproduce, install google translator as the only translator, make sure quick translation is enabled and then attempt to translate a job in the UI. It fails with a "The job has no provider assigned." exception, leaving the job in a broken, undefined state.

This is regression was caused with the refactoring for continuous translation and we have a single code base now for translating a whole job and translating some items, so we get the items and then the job again from them.

Proposed resolution

Save the job before completing quick-checkout.

Remaining tasks

User interface changes

API changes

Data model changes

Comments

Berdir created an issue. See original summary.

berdir’s picture

StatusFileSize
new686 bytes

Not sure how to write a test for this, maybe by loading and accessing the job translator in the test translator plugin.

berdir’s picture

Status: Needs review » Needs work
Issue tags: +Needs tests
berdir’s picture

Status: Needs work » Needs review
StatusFileSize
new1.28 KB

Test translator doesn't use that method for normal translation. Lets see what happens if we do this...

berdir’s picture

Issue tags: -Needs tests
StatusFileSize
new1.84 KB
new2.51 KB

This should work as a test-only patch.

The last submitted patch, 5: save-fails-2842321-5-test-only.patch, failed testing.

  • Berdir committed 729744a on 8.x-1.x
    Issue #2842321 by Berdir: Quick checkout can fail if a translator loads...
berdir’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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