improve extend query function

2gis / slot

JavaScript фреймворк, позволяющий быстро и просто создавать модульные изоморфные приложения.

http://2gis.github.io/slot

BSD 2-Clause "Simplified" License

86 stars 10 forks source link

Closed RrredHead closed 8 years ago

RrredHead commented 8 years ago

On server-side new location params protocol, host, hostname can be undefined. So, this cause wrong new location url.

ilyapro commented 8 years ago

мне кажется в этой функции (не в этом ПРе) нарушена логика

if (!urlOld || !urlNew) {
    if (!urlOld) return urlNew;
    if (!urlNew) return urlOld;
    return '';
}

скорей всего имелось в виду:

if (!urlOld && !urlNew) return '';
if (!urlOld) return urlNew;
if (!urlNew) return urlOld;

RrredHead commented 8 years ago

@ilyapro Действительно. Очень похоже, что ты прав. Поправил.

ilyapro commented 8 years ago

:+1:

ovvost commented 8 years ago

:+1:

RrredHead commented 8 years ago