search

corenova / yang-js

YANG parser and composer
Apache License 2.0
56 stars 18 forks source link

Property.merge does NOT work if there is content inside #76

Closed quantang closed 5 years ago

quantang commented 5 years ago

It looks like the Property.merge does nothing when there is content inside. Is it expected?

      merge: (value, opts={ replace: true, suppress: false }) ->
        opts.replace ?= true
        return @set value, opts unless @content?

        return this
sekur commented 5 years ago

Nope, another bug. :-) On Thu, Oct 11, 2018 at 5:55 PM Quan Tang notifications@github.com wrote:

It looks like the Property.merge does nothing when there is content inside. Is it expected?

  merge: (value, opts={ replace: true, suppress: false }) ->
    opts.replace ?= true
    return @set value, opts unless @content?

    return this

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/corenova/yang-js/issues/76, or mute the thread https://github.com/notifications/unsubscribe-auth/AA6KbCrS10ffvDiw_553WGnKJ1DT9UXCks5uj-hsgaJpZM4XYm98 .

-- Peter Lee Chief Executive Officer Corenova Technologies, Inc. +1-310-400-6450 linkedin.com/in/peter-k-lee