Fix adding users to quotes and invoices

[pierredup]

Fixes #816

[codecov]

Codecov Report

Patch coverage: 56.73% and project coverage change: -0.04 ⚠️

Comparison is base (5db4606) 50.60% compared to head (4b495e3) 50.57%.

Additional details and impacted files

@@             Coverage Diff              @@
##              2.2.x     #819      +/-   ##
============================================
- Coverage     50.60%   50.57%   -0.04%     
- Complexity     2201     2208       +7     
============================================
  Files           439      439              
  Lines          8303     8240      -63     
============================================
- Hits           4202     4167      -35     
+ Misses         4101     4073      -28     

Flag	Coverage Δ
unittests	50.57% <56.73%> (-0.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
src/ApiBundle/ApiTokenManager.php	100.00% <ø> (ø)
...le/Event/Listener/AuthenticationSuccessHandler.php	0.00% <ø> (ø)
.../Normalizer/AdditionalContactDetailsNormalizer.php	94.73% <ø> (ø)
...undle/Serializer/Normalizer/DiscountNormalizer.php	100.00% <ø> (ø)
src/ClientBundle/Action/Add.php	0.00% <ø> (ø)
src/ClientBundle/Action/Ajax/Address/Edit.php	0.00% <ø> (ø)
src/ClientBundle/Action/Ajax/Contact/Add.php	0.00% <ø> (ø)
src/ClientBundle/Action/Ajax/Contact/Edit.php	0.00% <ø> (ø)
src/ClientBundle/Action/Ajax/Delete.php	0.00% <ø> (ø)
src/ClientBundle/Action/Edit.php	0.00% <ø> (ø)
... and 172 more

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[hedho]

The setup now is all okay it asks to be generated a new user.

ERROR You need to select at least 1 user to attach to the Invoice
ERROR This value should not be blank.

Is still present eventhough the Client is selected!

[pierredup]

@hedho this fix isn't part of any release yet, it will be included once 2.2.3 is released