Opened 4 years ago

Closed 3 years ago

Last modified 3 years ago

#6012 closed defect (fixed)

Handle default choices for redeclare as default choices for parameters

Reported by: casella Owned by: adrpo
Priority: blocker Milestone: 1.16.0
Component: OMEdit Version:
Keywords: Cc: adrpo

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 casella 4 years ago.

Download all attachments as: .zip

Change History (5)

Changed 4 years ago by casella

comment:1 Changed 4 years ago by adrpo

  • Owner changed from adeas31 to adrpo
  • Status changed from new to accepted

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

comment:2 Changed 4 years ago by casella

  • Milestone changed from 1.16.0 to 1.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 Changed 3 years ago by adrpo

  • Resolution set to fixed
  • Status changed from accepted to closed

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

comment:4 Changed 3 years ago by casella

  • Milestone changed from 1.17.0 to 1.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.