[engine] MuranoPL polymorphism does not work is upcasted to superclass

Bug #1314618 reported by Alexander Tivelkov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Murano
Fix Released
High
Stan Lagun

Bug Description

If some derived class executes the method of its superclass (using $.super($.someMethod)), and that method attempts to call some virtual method (e.g. a method overridden in the derived class), then polymorphism does not work: a base version of method is executed instead of derived one

summary: - MuranoPL: Polymorphism does not work is upcasted to base class
+ MuranoPL: Polymorphism does not work is upcasted to superclass
Timur Sufiev (tsufiev-x)
summary: - MuranoPL: Polymorphism does not work is upcasted to superclass
+ [engine] MuranoPL polymorphism does not work is upcasted to superclass
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to murano-api (master)

Fix proposed to branch: master
Review: https://review.openstack.org/94144

Changed in murano:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to murano (master)

Reviewed: https://review.openstack.org/94144
Committed: https://git.openstack.org/cgit/stackforge/murano/commit/?id=c6b5887c754b9d97f578a6e37587571d3e387823
Submitter: Jenkins
Branch: master

commit c6b5887c754b9d97f578a6e37587571d3e387823
Author: Stan Lagun <email address hidden>
Date: Mon May 19 11:25:18 2014 +0400

    Restore identity of upcasted MPL object on method call

    When base class method was invoked using case/super method
    polymorphic method resolution mechanism didn't worked as
    expected. Now object identity is restored after DSL finds
    base class method and polymorphic calls work as they should

    Change-Id: I670ffaf5b28a60a3d269e85d25e0dfc7e164ee15
    Closes-Bug: #1314618

Changed in murano:
status: In Progress → Fix Committed
ruhe (ruhe)
Changed in murano:
status: Fix Committed → Fix Released
Changed in murano:
milestone: juno-1 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.