Баг или фича options.state about descript

[pasaran]

А если сделать result: state.items то что получится?

On Fri, Dec 19, 2014 at 11:33 PM, chestozo [email protected] wrote:

Вот для такой ручки:

de.object({
one: de.object(
{ items: { item: [ 'one', 'two' ] } },
{ state: { 'items': '.items.item' } }
),

two: de.object(
    { items: { item: [ 'three', 'four' ] } },
    { state: { 'items': '.items.item' } }
)

}, {
result: 'state.items'
})

Результатом будет:

[
"one",
"two",
"three",
"four"
]

Это баг или фича?

Мне кажется, это баг.
Один из state.items-ов должен был в итоге затереть другой state.items.

/cc @pasaran https://github.com/pasaran @edoroshenko
https://github.com/edoroshenko

—
Reply to this email directly or view it on GitHub
#105.

[chestozo]

Так у меня так и есть result: 'state.items'...

[pasaran]

В смысле result: 'state'

On Monday, December 22, 2014, chestozo [email protected] wrote:

Так у меня так и есть result: 'state.items'...

—
Reply to this email directly or view it on GitHub
#105 (comment).

[chestozo]

Будет так:

{
    "items": [
        "one",
        "two",
        "three",
        "four"
    ]
}

[edoroshenko]

там используется merge, он так работает. Баг это, или фича - вопрос философский. Но переделывать это место я бы не стал, лучше разрулить этот редкий кейс кастомной функцией. Это не то место, где можно с лёгкостью ломать совместимость

[chestozo]

А зачем там merge? Вы этим где-то пользуетесь осознанно?
Я это уже разрулил, да, просто поведение было неожиданное.

[pasaran]

Хмм. Я ничего про merge не помню, если честно )

[edoroshenko]

https://github.com/pasaran/descript/blob/master/lib/de.block.js#L337