Problem/Motivation
When we try to review a job from the job managment in /admin/tmgmt/jobs and then validate html tags from a translation that contains html tags, we get an error message about reciving an array instead of a string.
Steps to Recreate:
1.- Request a translation
2.- Go to jobs and pick manage in the dropdown.
3.- Select view/review in the job items.
4.- try to validate tags from an article with html tags.
Proposed resolution
If we get an array make it an string.
add tests
Remaining tasks
create patch, Review and commit.
User interface changes

Data model changes
Original report by [username]
Comments
Comment #2
juanse254 commentedHaving a hard time with tests, for some reason is failing locally and passing with fix. But simpletest doesnt seem to have the error. Feedback apreciated.
Comment #3
juanse254 commentedComment #4
miro_dietikerAh yeah, this is only with wysiwyg fields. They have that additional array nesting.
This really needs tests.
Comment #5
juanse254 commentedThis should be it.
:)
Comment #11
LKS90 commentedWhat if $source or $translation is an empty array?
The Jenkins build ran after the issue was committed, I guess they aren't retested like the testbot ones.
Edit: I just found out why I insisted on tests: https://www.drupal.org/node/2538534#comment-10227397 :D
Comment #12
juanse254 commentedThe test is right there, comment 5. Fixed the issue of empty array.
Comment #13
juanse254 commentedComment #16
juanse254 commentedComment #17
LKS90 commentedUnrelated changes (that don't do anything)?
Comment #18
juanse254 commentedRight, this is it :).
Comment #19
miro_dietikerYeah, looks fine now.
But tests pass before and still passing. So we need better test coverage for this case.
Critical because this is a repeated occurrence of the string VS array issue and leads to a fatal situation.
Comment #22
juanse254 commentedThis is the test only.
Comment #23
juanse254 commentedThis is the whole fix +tests.
Comment #28
miro_dietikerThx, committed!