Project

General

Profile

bug #9461

Filter in vaadin distribution editor does not work

Added by Andreas Müller 8 months ago. Updated 8 months ago.

Status:
Resolved
Priority:
New
Category:
cdm-vaadin
Target version:
Start date:
02/09/2021
Due date:
% Done:

50%

Severity:
normal
Found in Version:

Description

Previously the filter in the distribution editor did throw an exception (see #7254 and #9457). This is not the case anymore. However, the filter does not seem to work.

I tested on http://test.e-taxonomy.eu/cdmserver/flora_cuba/app/distribution with area filter on Cuba, West Cuba and Central Cuba and filter "Ancistranthus harpo*" or only "Ac*" but nothing appeared in the "Taxonomy" box


Related issues

Related to Edit - bug #7254: Filter in vaadin distribution editor throws exception Worksforme 02/12/2018
Related to Edit - bug #9457: Distribution editor: Taxon filter broken due to invalid HQL in TaxonNodeServiceImpl.getUuidAndTitleCache() Closed 02/08/2021

Associated revisions

Revision e6d5c3e2 (diff)
Added by Andreas Müller 8 months ago

ref #9457 , ref #9461 fix incorrect handling of AND/OR logic in doubtfulPattern handling in TaxonNodeDaoHibernateImpl.getUuidAndTitleCache

Revision 4672fa5f (diff)
Added by Andreas Müller 8 months ago

ref #9461 try to fix not working filter by using dto not dtos

Revision 15647253 (diff)
Added by Katja Luther 8 months ago

add test for uuidAndTitleCache in TaxonNodedao and code cleau

History

#1 Updated by Andreas Müller 8 months ago

  • Related to bug #7254: Filter in vaadin distribution editor throws exception added

#2 Updated by Andreas Müller 8 months ago

  • Related to bug #9457: Distribution editor: Taxon filter broken due to invalid HQL in TaxonNodeServiceImpl.getUuidAndTitleCache() added

#3 Updated by Andreas Müller 8 months ago

  • Status changed from New to Resolved
  • Assignee changed from Andreas Müller to Andreas Kohlbecker
  • Target version changed from Unassigned CDM tickets to Release 5.20
  • % Done changed from 0 to 50

The last commit seems to fix this issue. Can you please review.

#4 Updated by Andreas Kohlbecker 8 months ago

  • Assignee changed from Andreas Kohlbecker to Andreas Müller

Andreas Müller wrote:

The last commit seems to fix this issue. Can you please review.

Code changes look good. As far as you have tested the UI successfully you can close this ticket.

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 40 MB)