Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#6012 closed defect (fixed)

Handle default choices for redeclare as default choices for parameters

Reported by: Francesco Casella Owned by: Adrian Pop
Priority: blocker Milestone: 1.16.0
Component: OMEdit Version:
Keywords: Cc: Adrian Pop

Description

Steps to reproduce the issue:

  • activate replaceable support
  • open the attached package in OMEdit
  • select S
  • double-click on m
  • click on the drop-down menu

The following three classes show up:

TestReplaceablePar.RM2
TestReplaceablePar.RM1
TestReplaceablePar.M.RM

The first two choices are ok. I find the third a bit weird. From what I understand that should be treated in the same way as the default value in a regular parameter input widget. So, I would expect to get one empty item as the first element in the drop-down list (corresponding to "no redeclaration") and if I click on that, I should then see the greyed-out default value in the widget.

Attachments (1)

TestReplaceablePar.mo (660 bytes ) - added by Francesco Casella 4 years ago.

Download all attachments as: .zip

Change History (5)

by Francesco Casella, 4 years ago

Attachment: TestReplaceablePar.mo added

comment:1 by Adrian Pop, 4 years ago

Owner: changed from Adeel Asghar to Adrian Pop
Status: newaccepted

This should have worked. I'll have a look.

comment:2 by Francesco Casella, 4 years ago

Milestone: 1.16.01.17.0

We can keep this for 1.17.0, since it also involves parameters. Please check #6127 for a parameter-less example.

comment:3 by Adrian Pop, 4 years ago

Resolution: fixed
Status: acceptedclosed

As far as I can tell this is fixed. Reopen if not.

comment:4 by Francesco Casella, 4 years ago

Milestone: 1.17.01.16.0

It is, sorry. I was using the test case with parameters, but parameters are actually not involved here.

Note: See TracTickets for help on using tickets.