Implement base class for DataStorageViewer
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	kalali
	Nov 17 2017, 11:41 AM

Description

As a basis for T23296 we need to implement a base widget that can be used to implement different views on the data storage.
Such a base widget (DataStorageViewer) should have access to the data storage. It should be possible to filter the nodes in the data storage (using node predicates).

The idea is to allow developers to create their own concrete data storage view widget, without having to manage data storage node events or model selection events etc. The developer should only customize the view by providing a QtAbstractItemView and a QtAbstractItemModel.
The base class provides functions that are called on data storage node events and transforms model selection events into a list of currently selected data nodes.
The base class provides a signal / slot mechanism to send / receive selected data nodes to / from its local workbench view.

Revisions and Commits

Restricted Differential Revision

Related Objects
Search...

Status	Assigned	Task
		Restricted Maniphest Task
Resolved	None	T26741 Finishing new node selection concept
Resolved	floca	T23721 Project "Astonishing Angelfish"
Resolved	kalali	T23751 Introduction of new selection concept
Resolved	floca	T23754 DataStorage/Relationship clean up/refactoring
Open	None	T23811 PatientViewPlugin ++
Resolved	kalali	T23749 Implement base class for DataStorageViewer

Event Timeline

kalali created this task.Nov 17 2017, 11:41 AM

kalali edited parent tasks, added: T23751: Introduction of new selection concept; removed: T23721: Project "Astonishing Angelfish".Nov 17 2017, 11:51 AM

kalali updated the task description. (Show Details)Nov 20 2017, 1:10 PM

floca added a parent task: T23811: PatientViewPlugin ++.Nov 22 2017, 5:51 PM

Pushed new branch T23749-Base-class-for-data-storage-viewer.

kalali mentioned this in rMITKdab8aa4b04e1: T23749: Initial commit of the data storage viewer base class.Dec 4 2017, 2:38 PM

Regarding a different use of this class:
Currently we want to use this AbstractDataStorageViewer as a class, that takes a view and a model from the developer and connects the signals and slots in a way, that selections of the view are correctly set from outside or propagated outwards. Additionally the AbstractDataStorageViewer is used to connect the Node-events from the DataStorage to member functions.
However, these member functions do not provide any implementation as the developer of a concrete DataStorageViewer should be free to implement the desired behavior.

This leaves me thinking about the responsibility and use of this AbstractDataStorageViewer.
It could be split up into two classes, one that connects a given model with the events from the data storage (needed to update the model information when something in the data storage changed). The second class is a connector for the selections (send and receive). Both classes could be used in a way, that the view/model and the data storage / node predicate is given to the classes, but a custom data storage viewer only needs to know about these two classes.
There is no need for subclassing. A custom data storage viewer that subclasses the AbstractDataStorageViewer (as it is intended now) has to define the model/view to be used anyway (and thus customize the viewer) and then add the qt-view to its layout or the like.

kalali raised the priority of this task from Wishlist to Normal.Dec 21 2017, 1:21 PM

I changed the design:
We now provide a QmitkAbstractDataStorageModel that needs to be sub-classed for a custom model. The abstract model already connects the data storage node events (add, remove, change) with protected pure virtual functions.
These functions have to be overwritten in the derived model class to provide a custom behavior.
It furthermore accepts a node predicate to filter the nodes in the data storage.

This still fits the task's description. However, we do not provide a full widget anymore, but rather define the base model to be used for this scenario. The developer can still define it's view and customize the base model to his or her needs.

However, the handling of the model selections are moved to the QmitkModelViewSelectionConnector-class (see new modifications in T23752).

kalali mentioned this in T23752: Create a connector class to make data storage viewers be able to provide and receive global data node selections.Jan 3 2018, 4:02 PM

kalali mentioned this in rMITK4c37f9ec94f1: Add local selection handling to the selection-connector-class.Jan 3 2018, 5:26 PM

kalali mentioned this in rMITKd61b682ec565: Merge branch 'T23752-Selection-connector-for-data-storage-view' into T23749….Jan 12 2018, 5:58 PM

kalali added a revision: Restricted Differential Revision.Jan 12 2018, 6:24 PM

kalali mentioned this in T23751: Introduction of new selection concept.Jan 29 2018, 11:52 AM