bug #9843
closedWhen adding new descriptions it should not be necessary to load the complete matrix
100%
Description
copied from #9807:
The current solution solves that the matrix is completely collapsed which is helpful. However the aim is to keep it expanded in the same state as it was before and to not reload the full matrix as this takes time. We only need to load the aggregation records and to add/update them where necessary.
Related issues
Updated by Katja Luther over 1 year ago
- Related to bug #9807: Matrix should not collapse after aggregation added
Updated by Andreas Müller over 1 year ago
- Tags set to additivity, matrix, aggregation
Updated by Katja Luther over 1 year ago
- Status changed from New to Feedback
- Assignee changed from Katja Luther to Andreas Müller
Therefore it is necessary that the aggregation result contains the uuids of the newly created or updated descriptions otherwise it is not possible to get only the updated/new descriptions.
For adding new specimendescriptions to the matrix it is already implemented that only the new descriptions are added to the matrix.
Updated by Andreas Müller over 1 year ago
- Related to feature request #9858: Add update result to aggregation algorithms added
Updated by Andreas Müller over 1 year ago
- Related to deleted (feature request #9858: Add update result to aggregation algorithms)
Updated by Andreas Müller over 1 year ago
- Blocked by feature request #9858: Add update result to aggregation algorithms added
Updated by Andreas Müller over 1 year ago
- Status changed from Feedback to In Progress
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 0 to 10
The blocking issue (#9858) is resolved now
Updated by Katja Luther over 1 year ago
- Subject changed from [Character Matrix] When adding new descriptions it should not be necessary to load the complete matrix to When adding new descriptions it should not be necessary to load the complete matrix
- Status changed from In Progress to Resolved
- Assignee changed from Katja Luther to Andreas Müller
This is fixed now, please review.
Updated by Andreas Müller about 1 year ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 10 to 40
Can you leave a short message what has been done and how/where it can be tested.
What is the expected new behaviour and in which context the new funcitonality is used/available? Etc. With the information given it is difficult to find a starting point for review.
Updated by Katja Luther about 1 year ago
- Assignee changed from Katja Luther to Andreas Müller
Andreas Müller wrote in #note-9:
Can you leave a short message what has been done and how/where it can be tested.
What is the expected new behaviour and in which context the new funcitonality is used/available? Etc. With the information given it is difficult to find a starting point for review.
New RowWrappers are created for updated descriptions only and these rowWrappers are added to the matrix. The review should test whether the matrix does not collapse after aggregation and that the updated descriptions are displayed correctly.
Updated by Andreas Müller about 1 year ago
- Target version changed from Release 5.38 to Release 5.29
Updated by Andreas Müller about 1 year ago
- Status changed from Feedback to Closed
- Assignee changed from Andreas Müller to Katja Luther
- % Done changed from 40 to 100
This seems to work as expected.