Feedback:

• Let's add a UI test to catch this regression in the future
  (Row evolution for a label with & is a key functionnality of Piwik since most URLs will have ampersand 👍 )

I've added ' and " to a fixture, so it will get used in the rowevolution ui test.
Unfortunately I was not able to easily update the OmniFixture. I used the command mentioned on https://github.com/piwik/piwik/tree/master/tests#keep-omnifixture-up-to-date

./console tests:setup-fixture OmniFixture --sqldump=OmniFixture-dump.sql

but it throws a fatal error. Tried to fix it, but that ended with some other exceptions...

@mattab @diosmosis would you mind checking why the command doesn't work anymore? It seems the Fixture classes are not loaded correctly...

@sgiehl there was already & in the UI test, does it mean this bug should have been caught (and was caught) by UI tests? did we maybe miss it when the screenshot changed?

No, the problem occurred with '. & was already covered.

This fixes the OmniFixture issues for me: #9262

Oh. Updating the OmniFixture makes a lot of UI tests fail... It's hard to check which ones are expected.

@sgiehl could you maybe update omnifixture in a separate PR and we can more easily check if changes are expected? I can help check screenshots if needed

Reverted the Omnifixture update. Will create a separate PR for that soon. Guess this one can be merged then?

@sgiehl there is still changes to omnifixture in the PR, is it expected?

Now it should be fine.

Works 👍