Opened 12 years ago

Closed 12 years ago

#82 closed Bug (fixed)

Editor type checking shall consider inheritance

Reported by: Matthäus Wander Owned by: Thomas Schmid
Priority: Important Milestone:
Component: AnotherEditor Keywords:
Cc:

Description

  • Add Comparators and ComparatorsTmp to your solution, build and run
  • Open ProjectSamples\TestWorkspaces\IComparable_Comparator_Test.cte
  • Note, that this sample consists of two different comparator plugins: "Comparators" is based on object input type and "ComparatorsTmp" is based on IComparable input.
  • Remove the connections to "object Comparators" and try to connect NumberInput with "IComparable ComparatorsTmp".
  • Expected: Int32 implements IComparable. It should be possible to connect Int32 Output property with IComparable Input property.
  • Actual: Editor prevents connection.

Note: defect #80 can be observed as well from the above steps: the icon of the top comparator is "=" after loading, though the operation is set to "<".

Change History (3)

comment:1 Changed 12 years ago by Matthäus Wander

Resolution: fixed
Status: newclosed

Thanks, fixed by Thomas in r586.

comment:2 Changed 12 years ago by Thomas Schmid

Resolution: fixed
Status: closedreopened

In version 586 an interface check were added. This works as follows: All interfaces of both types that should be connected are checked. If they have at least one combined interface the connection is allowed. This leads to the problem that e.g. Int32 can be connected to string, double, bool etc., because all types implement IComparable. Check should be changed as follows: If one type is an interface and the other type implements this interface the connection is allowed (Ohter cases were connections are allowed not mentioned here).

comment:3 Changed 12 years ago by Thomas Schmid

Resolution: fixed
Status: reopenedclosed

(In [681]) - fixed #82

Note: See TracTickets for help on using tickets.