Project

General

Profile

Actions

bug #9843

closed

When adding new descriptions it should not be necessary to load the complete matrix

Added by Katja Luther over 2 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
New
Assignee:
Category:
taxeditor
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Severity:
normal
Found in Version:

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

Related to EDIT - bug #9807: Matrix should not collapse after aggregationClosedKatja Luther

Actions
Blocked by EDIT - feature request #9858: Add update result to aggregation algorithmsClosedAndreas Müller

Actions
Actions #1

Updated by Katja Luther over 2 years ago

  • Related to bug #9807: Matrix should not collapse after aggregation added
Actions #2

Updated by Andreas Müller over 2 years ago

  • Tags set to additivity, matrix, aggregation
Actions #3

Updated by Katja Luther over 2 years 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.

Actions #4

Updated by Andreas Müller over 2 years ago

Actions #5

Updated by Andreas Müller over 2 years ago

Actions #6

Updated by Andreas Müller over 2 years ago

Actions #7

Updated by Andreas Müller over 2 years 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

Actions #8

Updated by Katja Luther over 2 years 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.

Actions #9

Updated by Andreas Müller about 2 years 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.

Actions #10

Updated by Katja Luther about 2 years 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.

Actions #11

Updated by Andreas Müller about 2 years ago

  • Target version changed from Release 5.45 to Release 5.29
Actions #12

Updated by Andreas Müller about 2 years 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.

Actions

Also available in: Atom PDF