Skip to content

CompositeProperty throws if TimeInterval's data is not a Property #9236

Open
@thw0rted

Description

@thw0rted

Sandcastle example

Browser: Chrome

Operating System: Win10

I started with the Sandcastle demo for Interpolation, thinking it would be an easy way to see how CompositeProperty works. (Incidentally, there is no current Sandcastle demo that uses this class.) It's throwing and the error message doesn't make it obvious why.

Try to pause around property.intervals.addInterval(interval); with the debugger and step through until you get to CompositeProperty#subscribeAll. eventHelper.add throws when interval.data.definitionChanged is undefined.

One of two things needs to happen: if data is an arbitrary value with no definitionChanged event, skip the subscribe step; OR, update the docs for CompositeProperty (or I guess TimeInterval? probably not?) to specify that data must be a Property.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions