ns.Model.paramsRewrite ломает последовательность параметров в ключе

[chestozo]

Пример:

ns.Model.define('ppp', {
    params: {
        'p1': null,
        'p2': null,
        'p3': null
    },

    paramsRewrite: function(params) {
        if (true) {
            params.p2 = 'yes'
        }
        return params;
    }
});

console.log(
    ns.Model.get('ppp', { p1: 1, p3: 3 }).key,
    ns.Model.get('ppp', { p1: 1, p2: 'yes', p3: 3 }).key
);

Выводится:

model=ppp&p1=1&p3=3&p2=yes // !
model=ppp&p1=1&p2=yes&p3=3

Ожидаю:

model=ppp&p1=1&p2=yes&p3=3
model=ppp&p1=1&p2=yes&p3=3

[chestozo]

/cc @vitkarpov

[vitkarpov]

А почему ты ожидаешь порядок? Ты где-то в тестах смотришь на ключ? Кажется, это плохой вариант, лучше через АПИ ключ получать

[chestozo]

ммм какое api?

[vitkarpov]

getKeyAndParams

[chestozo]

так это оно же, просто оно внутри вызывается

[chestozo]

https://github.com/yandex-ui/noscript/blob/master/src/ns.model.js#L69

[vitkarpov]

Я имею ввиду, что в ключе вообще может быть любая упячка — главное взять хеш параметров и закодировать в ключе и наоборот: взять ключ и раскодировать в параметры

[vitkarpov]

Т.е. нельзя завязываться на определенный вид ключа

[vitkarpov]

Разве нет?

[chestozo]

в примере в ключе - одни и те же параметры, но ключ - разный
кажется, так быть не должно

[chestozo]

я просто хочу, чтобы ns.Model.get({ p1: 1, p2: 2 }) выдавал всегда тоже самое, что и ns.Model.get({ p2: 2, p1: 1 }). А тут это немного ломается, кажется, не?

[vitkarpov]

Ну да, я теперь понял о чем ты. Вопрос в том, создаются ли реально разные экземпляры моделей или нет? Я клоню к тому, что может если внутри мы пользуемся этим же АПИ и это не влияет на функциональность, то может и не надо (но сбивает с толку, это да)

[chestozo]

Да, разный ключ - разные экземпляры (

[chestozo]

Во, Саша нарыл #337

[chestozo]

Похоже, сортировка нужна только, если есть paramsRewrite )
https://github.com/yandex-ui/noscript/blob/master/src/ns.model.js#L839-L842
В остальных местах - ключ строится по pNames - т.е. по одной и той же последовательности.