Convert ports before parsing.

[piotrowski]

Refactor code to allow mixing notation with -p
Closes #1962

Signed-off-by: Aleksander Piotrowski apiotrowski312@gmail.com

- What I did
After refactoring and bug fixing. Docker provides better error message with invalid port:
docker run --rm -p 8080:87888888 busybox

and works with mixed notation like:
docker run --rm -p 8080:80 -p published=1500,target=444 busybox

- How I did it
I changed parsePortOpts, before it was iterating over all -p args and assuming that all of them are in the same notation. My fix is to iterate over all args and change every with = to short notation and then parse it.

- How to verify it
docker run --rm -p 8080:80 -p published=1500,target=444 busybox
docker run --rm -p 8080:87888888 busybox

- Description for the changelog
Fix a bug with parsing -p flag with mixed notation

- A picture of a cute animal (not mandatory but encouraged)

[codecov-io]

Codecov Report

Merging #1965 into master will increase coverage by 0.06%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master    #1965      +/-   ##
==========================================
+ Coverage   56.72%   56.79%   +0.06%     
==========================================
  Files         310      310              
  Lines       21800    21802       +2     
==========================================
+ Hits        12367    12383      +16     
+ Misses       8518     8503      -15     
- Partials      915      916       +1

[thaJeztah]

Thank you for contributing, and apologies for the delay: the original branch or repository for this PR was removed, so I carried your change in #2251, keeping your original sign-off. Thanks!