Hypertopic / Porphyry

Corpus analyses confrontation
https://hypertopic.org/porphyry
GNU Affero General Public License v3.0
21 stars 165 forks source link

The building map should be disabled when empty #560

Closed benel closed 2 years ago

benel commented 2 years ago

Description

What is the valuable outcome that cannot be achieved because of this bug?

With the building map extension enabled (visitMap: true), when a location is selected (spatial: xxxxx), items that correspond to the pattern /\w+ \d+/ are placed around the map according to the CNRS rules for numbering stain glasses in a church.

But some items may have good reasons for not matching the pattern: for example, they are not necessarily in a church (e.g. Société académique de l'Aube, Musée de Vauluisant).

For which stakeholder (people, role, project, domain) is it important?

To be defined.

Reproduction scenario

Steps to reproduce the behavior:

  1. Open the vitrauxportfolio.
  2. Select spatial: Société académique de l'Aube, Troyes
  3. The map is "loading" endlessly.
Capture d’écran 2022-02-22 à 11 54 20

Expected behavior

The default index should be shown instead.

Capture d’écran 2022-02-22 à 11 54 55

Environment

Workaround

Solution you found to avoid the problem.

Deliverables status

Phase 1

Phase 2

maxime-huyghe commented 2 years ago

The scenario and test are already done (portfolio_search.feature:5). The problem arises when we set the portfolio>vitraux>visitMap config value to true.

maxime-huyghe commented 2 years ago

Stratégie d'implémentation

Quelle partie du code sera impactée (classes, méthodes) ? Utilisez les React DevTools (extensions de Chrome ou Firefox) pour faire le lien entre la structure de la page et celle du code.

Le Portfolio sera impacté, car c'est lui qui définit les conditions d'affichage de la VisitMap.

Y aura-t-il des difficultés d'un point de vue algorithmique ? Avez-vous des pistes pour les résoudre ?

ø

Les données dont vous avez besoin sont-elles déjà chargées par la page ?

Les données dont nous avons besoin (est-ce qu'un plan est présent) ne sont pas chargées par la page.

Doit-on intégrer des services extérieurs (ex : cartographie) ? De quelles fonctionnalités en particulier aura-t-on besoin ? Est-ce que le service choisi les propose ou doit-on passer chez un concurrent ? Comment s'utilisent ces fonctionnalités ? avec quels paramètres ?

ø

benel commented 2 years ago

Fixed by 2be6c217b8c1e94e599410f48f8693ef2210d4c3.