Use PropertyWidgets in Custom Dialogs

[PeterParker]

This issue was originally TRAC 9149

This ticket is blocked by :

• TRAC2480

The dynamically-generated algorithm dialogs use PropertyWidget objects. The customised algorithm dialogs (and perhaps others) should be made to use them too. This will make things more consistent, both visually and within the code, it will avoid code duplication, and will also allow make the http://trac.mantidproject.org/mantid/ticket/2480 functionality more widespread.

''No work required'':

• SmoothNeighbours - (Already uses PropertyWidgets fully)
• LoadDAE - (Deleted algorithm)
• LOQScriptInput - (Deleted algorithm)

''Work required'':

• CatalogPublish
• ConvertTableToMatrixWorkspace
• CreateSampleShape
• Fit
• LoadAscii
• Load
• LoadInstrument
• LoadRaw
• PlotAsymmetryByLogValue
• BinMD - (SlicingAlgorithmDialog)
• SliceMD - (SlicingAlgorithmDialog)
• StartLiveData

[PeterParker]

@NickDraper (2014-06-23T08:01:10):
Moved to Backlog at the code freeze of release 3.2

---

@PeterParker (2015-03-15T18:25:09):
Refs http://trac.mantidproject.org/mantid/ticket/9149 - Remove CustomDialog for CatalogPublish algorithm.

Instead use IPropertySetting objects to handle property complexities.

Eagle-eyed code reviewers will note that these changes don't work very
well with composite catalogs, but neither did the custom algorithm dialog
in its previous form. This will need to be revisited when composite
catalogs are finished. In particular, what should be the allowed values
of the InvestigationNumber property given multiple session IDs? How
should the allowed values (as well as the default value) of the SessionID
property change as well?

Also, for now, "FileName" and "InputWorkspace" are back to being on two
separate rows in the dialog.

1ea0c96

---

@PeterParker (2015-03-15T18:25:09):
Refs http://trac.mantidproject.org/mantid/ticket/9149 - WIP. Needs rebasing.

c090365

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. If you feel this is incorrect please comment to keep it alive, with a reason why.

To prevent closure, e.g. for long-term planning issues, add the "Never Stale" label.

[stale]

This issue has been closed automatically. If this still affects you please re-open this issue with a comment so we can look into resolving it.