• 
      

    Update base objects to use static and merge for configurable attributes.

    Review Request #12909 — Created March 22, 2023 and submitted — Latest diff uploaded

    Information

    Spina
    master

    Reviewers

    Starting in Spina 2, BaseModel, BaseView, and BaseCollection's
    configurable attributes are now defined as static, allowing them to be
    accessed outside of instances. This is important for subclasses.

    These make use of the new prototypeAttrs option, which ensures they'll
    be accessible on the prototype, allowing Backbone to behave as normal
    and for instances to continue accessing using this.

    Types for both the static and instance/prototype versions are set on
    these base classes. Since static is incompatible with generics, we use
    the most base type for these there, but the ones on the prototype are
    typed as expected. Subclasses may still define types for their versions.

    Some attributes (events, modelEvents, and defaults) are also set
    up to auto-merge between parents/subclasses. This means that subclasses
    no longer have to manually call _.defaults or _.extends, passing in
    the parent class's attribute (but at the same time, it won't hurt if
    they still do). This will keep things more maintainable going forward.

    Porting existing code in Review Board to set the appropriate attributes
    as static, and verified that instances and subclasses could properly
    access and extend these.

    Commits

    Files