Handled duplicates, replaced description with category

html_items.py

@@ -110,7 +110,7 @@ def create_poi_card_selectable(poi: dict, is_selected: bool = False) -> str:
         # Darker text colors for selected state
         title_color = "#2c5530"
         location_color = "#4a5c4d"
-        description_color = "#5a6b5d"
+        category_color = "#5a6b5d"
     else:
         border_color = "#ddd"
         background_color = "white"
@@ -119,7 +119,7 @@ def create_poi_card_selectable(poi: dict, is_selected: bool = False) -> str:
         content_bg = "white"
         title_color = "#333"
         location_color = "#666"
-        description_color = "#777"
+        category_color = "#777"
 
     # Create compact sensory features display
     sensory_html = ""
@@ -147,7 +147,8 @@ def create_poi_card_selectable(poi: dict, is_selected: bool = False) -> str:
         """
 
     # Escape single quotes in POI name for JavaScript
-    poi_name_escaped = poi["name:token_seq"].replace("'", "\\'")
+    poi_id_escaped = poi["poi_id:token"].replace(" ", "-").replace("'", "").replace('"', "")
+    poi_category = ' - '.join(poi["category:token_seq"]).replace('_', ' ')
     
     html = f"""
     <div class="poi-card" data-poi-name="{poi["name:token_seq"]}" 
@@ -161,7 +162,7 @@ def create_poi_card_selectable(poi: dict, is_selected: bool = False) -> str:
                 position: relative;
                 overflow: hidden;
                 box-shadow: 0 2px 8px rgba(0,0,0,0.1);"
-         onclick="document.getElementById('poi-btn-{poi_name_escaped.replace(" ", "-")}').click();"
+         onclick="document.getElementById('poi-btn-{poi_id_escaped}').click();"
          onmouseover="this.style.transform='scale(1.02)'; this.style.boxShadow='0 4px 12px rgba(0,0,0,0.15)';" 
          onmouseout="this.style.transform='scale(1)'; this.style.boxShadow='0 2px 8px rgba(0,0,0,0.1)';">
         
@@ -185,8 +186,8 @@ def create_poi_card_selectable(poi: dict, is_selected: bool = False) -> str:
             <p style="margin: 0 0 8px 0; color: {location_color}; font-size: 14px; font-weight: 500;">
                 📍 {poi["address:token_seq"]}
             </p>
-            <p style="margin: 0 0 12px 0; color: {description_color}; font-size: 14px; line-height: 1.4;">
-                {poi["description:token_seq"][:120]}{"..." if len(poi["description:token_seq"]) > 120 else ""}
+            <p style="margin: 0 0 12px 0; color: {category_color}; font-size: 14px; line-height: 1.4;">
+                {'🏷️ ' + poi_category if poi_category else ''}
             </p>
             
             {sensory_html}
@@ -203,14 +204,14 @@ def create_poi_card_selectable(poi: dict, is_selected: bool = False) -> str:
     return html
 
 
-def create_poi_cards_list(pois: list, selected_names: set) -> str:
+def create_poi_cards_list(pois: list, selected_pois: set) -> str:
     if not pois:
         return "<div style='text-align:center; color:#666; margin: 40px; padding: 40px; background: #f8f9fa; border-radius: 12px;'><h3>🔍 Nessun punto di interesse trovato</h3><p>Prova con termini di ricerca diversi</p></div>"
 
     # Genera HTML per i POI filtrati
     poi_cards_html = []
     for poi in pois:
-        is_selected = poi.get("name:token_seq") in selected_names
+        is_selected = (poi.get("poi_id:token"), poi.get("name:token_seq")) in selected_pois
         card_html = create_poi_card_selectable(poi, is_selected)
         poi_cards_html.append(card_html)
 
@@ -219,7 +220,7 @@ def create_poi_cards_list(pois: list, selected_names: set) -> str:
         {"".join(poi_cards_html)}
     </div>
     <div style='text-align: center; color: black; margin-top: 20px; padding: 15px; background: #f0f0f0; border-radius: 8px;'>
-        <strong style='color: black;'>Trovati {len(pois)} punti di interesse</strong> • <span style='color: #7FB77E;'>{len(selected_names)} selezionati</span>
+        <strong style='color: black;'>Trovati {len(pois)} punti di interesse</strong> • <span style='color: #7FB77E;'>{len(selected_pois)} selezionati</span>
     </div>
     """
 

main.py

@@ -169,7 +169,7 @@ def create_main_app(self):
         # Stati globali dell'applicazione
         app_user_id = gr.State("")
         current_page = gr.State("login")  # "login", "poi_selection", "sensory_config", "recommendations"
-        selected_poi_names = gr.State(set())
+        current_selected_pois = gr.State(set())
         recommendations_state = gr.State([])
         selected_poi_index = gr.State(-1)
         completed_survey_pois = gr.State(set())
@@ -250,14 +250,15 @@ def create_main_app(self):
                     # Create hidden buttons for each POI
                     poi_buttons = {}
                     for poi in pois_list:
+                        poi_id = poi["poi_id:token"]
                         poi_name = poi["name:token_seq"]
-                        safe_name = (
-                            poi_name.replace(" ", "-").replace("'", "").replace('"', "")
+                        safe_id = (
+                            poi_id.replace(" ", "-").replace("'", "").replace('"', "")
                         )
                         button = gr.Button(
-                            poi_name, visible=False, elem_id=f"poi-btn-{safe_name}"
+                            poi_name, visible=False, elem_id=f"poi-btn-{safe_id}"
                         )
-                        poi_buttons[poi_name] = button
+                        poi_buttons[(poi_id, poi_name)] = button
 
             with gr.Column(
                 visible=False, elem_classes="sensory-config-main-container"
@@ -442,19 +443,19 @@ def create_main_app(self):
                 login_status,  # login_status
             )
 
-        def handle_poi_click(poi_name, selected_names, query):
+        def handle_poi_click(poi_id, poi_name, selected_pois, query):
             """Handle POI click - toggle selection"""
             print(f"🔥 POI Button clicked: {poi_name}")
-            return toggle_poi_selection(poi_name, selected_names, query)
-        
-        def handle_proceed_to_sensory_config(user_id, selected_names):
+            return toggle_poi_selection(poi_id, poi_name, selected_pois, query)
+
+        def handle_proceed_to_sensory_config(user_id, selected_pois):
             """Handle proceeding to next step"""
-            if not selected_names:
+            if not selected_pois:
                 gr.Warning("⚠️ Seleziona la card di almeno un Punto di Interesse per continuare!")
                 return
 
-            update_user_selected_pois(user_id, selected_names)
-            gr.Info(f"✅ Procedendo con {len(selected_names)} Punti di Interesse selezionati")
+            update_user_selected_pois(user_id, selected_pois)
+            gr.Info(f"✅ Procedendo con {len(selected_pois)} Punti di Interesse selezionati")
             return navigate_to_sensory_config()
 
         def navigate_to_poi_selection():
@@ -537,28 +538,28 @@ def create_main_app(self):
         # Dynamic search - triggers on every keystroke
         search_input.change(
             fn=search_pois,
-            inputs=[search_input, selected_poi_names],
+            inputs=[search_input, current_selected_pois],
             outputs=[poi_display],
         )
 
         # Set up click handlers for each POI button
-        for poi_name, button in poi_buttons.items():
+        for (poi_id, poi_name), button in poi_buttons.items():
             button.click(
-                fn=partial(handle_poi_click, poi_name),
-                inputs=[selected_poi_names, search_input],
-                outputs=[selected_poi_names, poi_display, selection_summary],
+                fn=partial(handle_poi_click, poi_id, poi_name),
+                inputs=[current_selected_pois, search_input],
+                outputs=[current_selected_pois, poi_display, selection_summary],
             )
 
         # Clear all selections
         clear_button.click(
             fn=clear_all_selections,
             inputs=[search_input],
-            outputs=[selected_poi_names, poi_display, selection_summary],
+            outputs=[current_selected_pois, poi_display, selection_summary],
         )
 
         continue_to_sensory_btn.click(
             fn=handle_proceed_to_sensory_config,
-            inputs=[app_user_id, selected_poi_names],
+            inputs=[app_user_id, current_selected_pois],
             outputs=[
                 login_page,
                 poi_selection_page,

utils/poi_search_utils.py

@@ -10,9 +10,11 @@ try:
     with open(os.path.join(RESOURCE_DIR, "turin_autism.item.json"), "r") as f:
         pois_list = json.load(f)
     for poi in pois_list:
-        poi["description:token_seq"] = poi.get("description", "")
-        poi["address:token_seq"] = poi.get("address", "")
-        poi["coordinates:float_seq"] = poi.get("coordinates", [])
+        for col in ["description:token_seq", "address:token_seq", "coordinates:float_seq"]:
+            if poi[col] is None:
+                poi[col] = ""
+        if poi["category:token_seq"] is None:
+            poi["category:token_seq"] = [""]
 except FileNotFoundError:
     with open(os.path.join(RESOURCE_DIR, "mock_data.json"), "r") as f:
         LOADED_MOCK_POIS = json.load(f)
@@ -20,7 +22,7 @@ except FileNotFoundError:
 check_image_url(pois_list)
 
 
-def search_pois(search_query: str, selected_names: set) -> str:
+def search_pois(search_query: str, selected_pois: set) -> str:
     """Cerca POI basandosi sulla query di ricerca e restituisce HTML"""
 
     # Filtra i POI basandosi sulla query
@@ -40,26 +42,26 @@ def search_pois(search_query: str, selected_names: set) -> str:
             ):
                 filtered_pois.append(poi)
 
-    return create_poi_cards_list(filtered_pois, selected_names)
+    return create_poi_cards_list(filtered_pois, selected_pois)
 
 
 def toggle_poi_selection(
-    poi_name: str, selected_names: set, current_query: str
+    poi_id, poi_name: str, selected_pois: set, current_query: str
 ) -> tuple[set, str, str]:
     """Alterna la selezione di un POI e aggiorna il display"""
 
     # Copia il set per evitare modifiche dirette
-    new_selected = selected_names.copy()
+    new_selected = selected_pois.copy()
 
-    print(f"DEBUG: Toggling POI '{poi_name}'")
-    print(f"DEBUG: Current selected: {selected_names}")
+    print(f"DEBUG: Toggling POI '{poi_name} ({poi_id})' selection")
+    print(f"DEBUG: Current selected: {selected_pois}")
 
-    if poi_name in new_selected:
-        new_selected.remove(poi_name)
-        print(f"DEBUG: Removed '{poi_name}' from selection")
+    if (poi_id, poi_name) in new_selected:
+        new_selected.remove((poi_id, poi_name))
+        print(f"DEBUG: Removed '{poi_name} ({poi_id})' from selection")
     else:
-        new_selected.add(poi_name)
-        print(f"DEBUG: Added '{poi_name}' to selection")
+        new_selected.add((poi_id, poi_name))
+        print(f"DEBUG: Added '{poi_name} ({poi_id})' to selection")
 
     print(f"DEBUG: New selected: {new_selected}")
 
@@ -73,13 +75,13 @@ def toggle_poi_selection(
     return new_selected, updated_html, summary
 
 
-def get_selection_summary(selected_names: set) -> str:
+def get_selection_summary(selected_pois: set) -> str:
     """Restituisce un riassunto dei POI selezionati"""
-    if not selected_names:
+    if not selected_pois:
         return "**Punti di Interesse selezionati: 0** - Seleziona i luoghi di tuo interesse cliccando sulle card"
 
-    names_list = list(selected_names)
-    return f"**Punti di Interesse selezionati: {len(selected_names)}** => {' - '.join(names_list[:3])}{' e altri...' if len(selected_names) > 3 else ''}"
+    names_list = [name for _, name in selected_pois]
+    return f"**Punti di Interesse selezionati: {len(names_list)}** => {' - '.join(names_list[:3])}{' e altri...' if len(names_list) > 3 else ''}"
 
 
 def clear_all_selections(current_query: str) -> tuple[set, str, str]: