UrlStateManagerService::pushState does not consider port

[intco]

Hi,

when running yoyo with a port different than 80 the following error show up in developer console:

DOMException: Failed to execute 'pushState' on 'History': 
A history state object with URL 'http://127.0.0.1/?count=1' cannot be created in a document 
with origin 'http://127.0.0.1:8132' and URL 'http://127.0.0.1:8132/'.

the problem is in the pushState method of UrlStateManagerService class

yoyo/src/yoyo/Services/UrlStateManagerService.php

Line 30 in d04eb38

$url = $parsedUrl['scheme'].'://'.$parsedUrl['host'].$parsedUrl['path'].($queryParams ? '?'.http_build_query($queryParams) : '');

The following fix the issue:

        $port = isset($parsedUrl['port']) ? (':'.$parsedUrl['port']) : '';
        $url = $parsedUrl['scheme'].'://'.$parsedUrl['host'].$port.$parsedUrl['path'].($queryParams ? '?'.http_build_query($queryParams) : '');

thank you for your time

[jreviews]

@intco Can you submit a PR request to the develop branch?

[intco]

PR created

thank you

[jreviews]

Thanks! I'll try to merge and tag a new release soon.