osate / osate2

Open Source AADL2 Tool Environment
http://osate.org
Eclipse Public License 2.0
36 stars 8 forks source link

Cannot edit property association with property editor #397

Closed reteprelief closed 9 years ago

reteprelief commented 10 years ago

I can add a property association to c component classifier with the PA+ button. When I invoke it again and select the same property it will add a second property association of that name. This should not happen. Note that the AADL Property Value viewer only shows the first property association with a given name. Instead it would be nice to have a way of editing the value of an existing property association - maybe by just double clicking in the AADL Property Value view on the value.

joeseibel commented 9 years ago

Additional enhancements for the property view:

joeseibel commented 9 years ago

David,

Lutz asked me to assign the remainder of this issue to you. Please look at the previous comment and you will see a list of items with many of them crossed out. The ones that are not crossed out are yet to be done.

ArslanIbnDaud commented 9 years ago
lwrage commented 9 years ago

The description for the first item is not quite accurate. The checkbutton should restrict the properties to those that are defined in either a predefined property set or a property set whose name is listed in a with statement in the current package. (Predefined property sets are in org.osate.workspace/resources/properties)

There is a sort button in the outline view. We should reuse its icon.

ArslanIbnDaud commented 9 years ago

Lutz says, via email:

In Osate, create a new aadl file ps.aadl:

property set ps is prop: aadlinteger applies to (system); end ps;

Create another file x.aadl:

package X public

with ps;

system S
end S;

end X;

Open x.aadl in the editor and click on the name S. Now the property view shows property sets, including ps. If you remove the 'with ps;' line from x.aadl, ps is still shown in the AADL property view. Whenever the new filter is enabled, ps should be shown if and only if the with clause is in the aadl source file.

With the filter turned on, and when the current aadl file contains no with clause at all, only predefined property sets should be shown in the property view.

ArslanIbnDaud commented 9 years ago

I should add that the 'imported properties / with statement' button is in fact very similar to the 'show undefined' button. The feature only becomes necessary when undefined properties are shown; it's prob useless when they are hidden.

lwrage commented 9 years ago

These are the remaining items

aaronvolkmann commented 9 years ago

I spoke with lwrange and we agreed that "sort alphabetically (property set names and property names)" was already implemented elsewhere.

aaronvolkmann commented 9 years ago

implemented "filter view by property sets occurring inwith statements" we may want to customize the icon in the future.