Fix: Suchergebnisse werden wieder korrekt angezeigt

CHANGELOG.md

@@ -5,6 +5,87 @@ Alle wichtigen Änderungen an diesem Projekt werden in dieser Datei dokumentiert
 Das Format basiert auf [Keep a Changelog](https://keepachangelog.com/de/1.0.0/),
 und dieses Projekt adhäriert zu [Semantic Versioning](https://semver.org/lang/de/).
 
+## [v1.2.9] - 2024-03-19
+### Geändert
+- Verbesserte Darstellung der Adress-Links mit Location- und Route-Icons
+- Korrektur des Docker-Port-Mappings für bessere Erreichbarkeit
+
+## [v1.2.8] - 2024-03-19
+### Geändert
+- Verbesserte Tag-Anzeige: Badge neben dem Teilen-Button mit farblicher Hervorhebung (blau für MEDISOFT, orange für MEDICONSULT)
+
+## [v1.2.7] - 2024-03-19
+### Geändert
+- Korrektur der Feldreihenfolge in den Suchergebnissen
+- Verbesserung der Tag-Anzeige
+
+## [v1.2.6] - 2024-03-19
+### Geändert
+- Verbesserte Darstellung der Kundennummern in den Suchergebnissen
+- Optimierte Anzeige der Kundennummern in der Detailansicht
+- Anpassung der Kundennummernberechnung für die MEDISOFT-Integration
+- Verbesserte Formatierung der Kundennummern in der Benutzeroberfläche
+
+## [v1.2.5] - 2024-03-19
+### Geändert
+- Verbesserte Darstellung der Kundennummern in den Suchergebnissen
+- Optimierte Anzeige der Kundennummern in der Detailansicht
+- Anpassung der Kundennummernberechnung für die MEDISOFT-Integration
+- Verbesserte Formatierung der Kundennummern in der Benutzeroberfläche
+
+## [v1.2.4] - 2024-03-19
+### Geändert
+- Verbesserte Darstellung der Kundennummern in den Suchergebnissen
+- Optimierte Anzeige der Kundennummern in der Detailansicht
+- Anpassung der Kundennummernberechnung für die MEDISOFT-Integration
+- Verbesserte Formatierung der Kundennummern in der Benutzeroberfläche
+
+## [v1.2.3] - 2024-03-19
+### Geändert
+- Verbesserte Darstellung der Kundennummern in den Suchergebnissen
+- Optimierte Anzeige der Kundennummern in der Detailansicht
+- Anpassung der Kundennummernberechnung für die MEDISOFT-Integration
+- Verbesserte Formatierung der Kundennummern in der Benutzeroberfläche
+
+## [v1.2.2] - 2024-03-19
+### Geändert
+- Verbesserte Darstellung der Kundennummern in den Suchergebnissen
+- Optimierte Anzeige der Kundennummern in der Detailansicht
+- Anpassung der Kundennummernberechnung für die MEDISOFT-Integration
+- Verbesserte Formatierung der Kundennummern in der Benutzeroberfläche
+
+## [v1.2.1] - 2024-03-19
+### Geändert
+- Verbesserte Darstellung der Kundennummern in den Suchergebnissen
+- Optimierte Anzeige der Kundennummern in der Detailansicht
+- Anpassung der Kundennummernberechnung für die MEDISOFT-Integration
+- Verbesserte Formatierung der Kundennummern in der Benutzeroberfläche
+
+## [v1.2.0] - 2024-03-19
+### Hinzugefügt
+- Neue Spalte "Tag" in der Kundendatenbank
+- Unterscheidung zwischen MEDISOFT und MEDICONSULT Kunden
+- Filteroption für die Anzeige von MEDISOFT oder MEDICONSULT Kunden
+- Verbesserte Darstellung der Kundennummern in den Suchergebnissen
+- Optimierte Anzeige der Kundennummern in der Detailansicht
+- Anpassung der Kundennummernberechnung für die MEDISOFT-Integration
+- Verbesserte Formatierung der Kundennummern in der Benutzeroberfläche
+
+## [v1.1.0] - 2024-03-19
+### Hinzugefügt
+- Neue Spalte "Tag" in der Kundendatenbank
+- Unterscheidung zwischen MEDISOFT und MEDICONSULT Kunden
+- Filteroption für die Anzeige von MEDISOFT oder MEDICONSULT Kunden
+
+## [v1.0.0] - 2024-03-19
+### Hinzugefügt
+- Erste Version der Anwendung
+- Grundlegende Suchfunktionalität
+- Anzeige von Kundendetails
+- Integration mit MEDISOFT
+- Responsive Design
+- IP-basierte Zugriffskontrolle
+
 ## [1.2.6] - 2024-03-19
 ### Geändert
 - Verbesserte Suchfunktion: Keine Ergebnisse mehr bei leeren Suchfeldern

README.md

@@ -1,35 +1,49 @@
-# Medi-Customers
+# medisoftware Kundensuche
 
-Eine Flask-basierte Webanwendung zur Verwaltung von Kundenkontakten für medizinische Einrichtungen.
+Eine moderne Webanwendung zur Suche und Verwaltung von Kundendaten, die MEDISOFT und MEDICONSULT Daten kombiniert.
 
 ## Features
 
-- Kundensuche nach verschiedenen Kriterien (Name, Ort, Kundennummer, etc.)
-- Direkte Links zu Kundendaten in medisoftware (für autorisierte IPs)
-- Telefonnummern-Links für autorisierte IPs
-- Adress-Links mit Google Maps Integration
-- IP-basierte Zugriffssteuerung
-- Responsive Design
+- 🔍 Echtzeit-Suche über alle Kundendaten
+- 📱 Responsive Design für alle Geräte
+- 🔒 IP-basierte Zugriffskontrolle
+- 🔗 Direkte Integration mit MEDISOFT
+- 🏥 Unterscheidung zwischen MEDISOFT und MEDICONSULT Kunden
+- 🎨 Farbliche Hervorhebung der Kundentypen (blau für MEDISOFT, orange für MEDICONSULT)
+- 📍 Verbesserte Adress-Links mit Location- und Route-Icons
+
+## Version
+
+Aktuelle Version: v1.2.9
 
 ## Installation
 
-1. Repository klonen:
+1. Klonen Sie das Repository:
 ```bash
 git clone https://gitea.elpatron.me/elpatron/medi-customers.git
 cd medi-customers
 ```
 
-2. Umgebungsvariablen einrichten:
+2. Erstellen Sie die erforderlichen Verzeichnisse:
 ```bash
-cp .env.example .env
-# Bearbeiten Sie die .env-Datei mit Ihren Einstellungen
+mkdir -p data
 ```
 
-3. Docker Container starten:
+3. Starten Sie die Anwendung mit Docker Compose:
 ```bash
-docker-compose up -d
+docker-compose up --build
 ```
 
+Die Anwendung ist dann unter `http://localhost:5000` erreichbar.
+
+## Entwicklung
+
+Die Anwendung ist in Python mit Flask entwickelt und verwendet SQLite als Datenbank. Das Frontend wurde mit HTML, CSS und JavaScript implementiert.
+
+## Lizenz
+
+Alle Rechte vorbehalten. © 2024 medisoftware
+
 ## Konfiguration
 
 Die Anwendung kann über folgende Umgebungsvariablen konfiguriert werden:
@@ -49,14 +63,6 @@ Die Anwendung unterstützt CIDR-Notation für IP-Bereiche. Beispiele:
 - Subnetz: 192.168.1.0/24
 - Größeres Netzwerk: 10.0.0.0/8
 
-## Version
-
-Aktuelle Version: v1.2.6
-
-## Lizenz
-
-Alle Rechte vorbehalten. © 2025 medisoftware
-
 ## API-Beispiele
 
 ### Suche nach Name

app.py

@@ -138,23 +138,28 @@ def import_csv():
 
 def search_customers():
     try:
-        q = request.args.get('q', '')
-        name = request.args.get('name', '')
-        ort = request.args.get('ort', '')
-        nummer = request.args.get('nummer', '')
-        plz = request.args.get('plz', '')
-        fachrichtung = request.args.get('fachrichtung', '')
-        operator = request.args.get('operator', 'or')
+        if request.method == 'POST':
+            data = request.get_json()
+            query = data.get('query', '')
+            tag = data.get('tag', 'medisoft')
+        else:
+            query = request.args.get('q', '')
+            name = request.args.get('name', '')
+            ort = request.args.get('ort', '')
+            nummer = request.args.get('nummer', '')
+            plz = request.args.get('plz', '')
+            fachrichtung = request.args.get('fachrichtung', '')
+            operator = request.args.get('operator', 'or')
+            tag = request.args.get('tag', 'medisoft')
             
         conn = get_db_connection()
         c = conn.cursor()
         
         # Basis-SQL-Query
-        query = '''
-            SELECT DISTINCT 
-                c.id,
-                c.name,
+        sql = '''
+            SELECT 
                 c.nummer,
+                c.name,
                 c.strasse,
                 c.plz,
                 c.ort,
@@ -168,68 +173,87 @@ def search_customers():
         '''
         params = []
         
-        # Suchbedingungen
-        conditions = []
-        if q:
-            search_terms = q.split()
-            if operator == 'and':
-                for term in search_terms:
-                    conditions.append('''
-                        (c.name LIKE ? OR c.nummer LIKE ? OR c.strasse LIKE ? 
-                        OR c.plz LIKE ? OR c.ort LIKE ? OR c.telefon LIKE ? 
-                        OR c.mobil LIKE ? OR c.email LIKE ? OR c.fachrichtung LIKE ?
-                        OR c.tag LIKE ?)
-                    ''')
-                    params.extend([f'%{term}%'] * 10)
-            else:
-                term_conditions = []
-                for term in search_terms:
-                    term_conditions.append('''
-                        (c.name LIKE ? OR c.nummer LIKE ? OR c.strasse LIKE ? 
-                        OR c.plz LIKE ? OR c.ort LIKE ? OR c.telefon LIKE ? 
-                        OR c.mobil LIKE ? OR c.email LIKE ? OR c.fachrichtung LIKE ?
-                        OR c.tag LIKE ?)
-                    ''')
-                    params.extend([f'%{term}%'] * 10)
-                conditions.append('(' + ' OR '.join(term_conditions) + ')')
+        if request.method == 'POST':
+            if query:
+                sql += ''' AND (
+                    c.name LIKE ? OR
+                    c.nummer LIKE ? OR
+                    c.strasse LIKE ? OR
+                    c.plz LIKE ? OR
+                    c.ort LIKE ? OR
+                    c.telefon LIKE ? OR
+                    c.mobil LIKE ? OR
+                    c.email LIKE ? OR
+                    c.fachrichtung LIKE ?
+                )'''
+                search_pattern = f'%{query}%'
+                params.extend([search_pattern] * 9)
+        else:
+            # Suchbedingungen für GET-Request
+            conditions = []
+            if query:
+                search_terms = query.split()
+                if operator == 'and':
+                    for term in search_terms:
+                        conditions.append('''
+                            (c.name LIKE ? OR c.nummer LIKE ? OR c.strasse LIKE ? 
+                            OR c.plz LIKE ? OR c.ort LIKE ? OR c.telefon LIKE ? 
+                            OR c.mobil LIKE ? OR c.email LIKE ? OR c.fachrichtung LIKE ?)
+                        ''')
+                        params.extend([f'%{term}%'] * 9)
+                else:
+                    term_conditions = []
+                    for term in search_terms:
+                        term_conditions.append('''
+                            (c.name LIKE ? OR c.nummer LIKE ? OR c.strasse LIKE ? 
+                            OR c.plz LIKE ? OR c.ort LIKE ? OR c.telefon LIKE ? 
+                            OR c.mobil LIKE ? OR c.email LIKE ? OR c.fachrichtung LIKE ?)
+                        ''')
+                        params.extend([f'%{term}%'] * 9)
+                    conditions.append('(' + ' OR '.join(term_conditions) + ')')
             
-        if name:
-            conditions.append('c.name LIKE ?')
-            params.append(f'%{name}%')
-        if ort:
-            conditions.append('c.ort LIKE ?')
-            params.append(f'%{ort}%')
-        if nummer:
-            conditions.append('c.nummer LIKE ?')
-            params.append(f'%{nummer}%')
-        if plz:
-            conditions.append('c.plz LIKE ?')
-            params.append(f'%{plz}%')
-        if fachrichtung:
-            conditions.append('c.fachrichtung LIKE ?')
-            params.append(f'%{fachrichtung}%')
+            if name:
+                conditions.append('c.name LIKE ?')
+                params.append(f'%{name}%')
+            if ort:
+                conditions.append('c.ort LIKE ?')
+                params.append(f'%{ort}%')
+            if nummer:
+                conditions.append('c.nummer LIKE ?')
+                params.append(f'%{nummer}%')
+            if plz:
+                conditions.append('c.plz LIKE ?')
+                params.append(f'%{plz}%')
+            if fachrichtung:
+                conditions.append('c.fachrichtung LIKE ?')
+                params.append(f'%{fachrichtung}%')
             
-        if conditions:
-            query += ' AND ' + ' AND '.join(conditions)
+            if conditions:
+                sql += ' AND ' + ' AND '.join(conditions)
         
-        c.execute(query, params)
+        # Füge Tag-Filter hinzu, wenn nicht 'all' ausgewählt ist
+        if tag != 'all':
+            sql += ' AND c.tag = ?'
+            params.append(tag)
+            
+        sql += ' ORDER BY c.name'
+        
+        c.execute(sql, params)
         results = c.fetchall()
         
-        # Formatiere die Ergebnisse
         formatted_results = []
         for row in results:
             customer = {
-                'id': row[0],
+                'nummer': row[0],
                 'name': row[1],
-                'nummer': row[2],
-                'strasse': row[3],
-                'plz': row[4],
-                'ort': row[5],
-                'telefon': row[6],
-                'mobil': row[7],
-                'email': row[8],
-                'fachrichtung': row[9],
-                'tag': row[10]
+                'strasse': row[2],
+                'plz': row[3],
+                'ort': row[4],
+                'telefon': row[5],
+                'mobil': row[6],
+                'email': row[7],
+                'fachrichtung': row[8],
+                'tag': row[9]  # Tag-Feld hinzugefügt
             }
             formatted_results.append(customer)
             
@@ -237,7 +261,7 @@ def search_customers():
         return jsonify(formatted_results)
         
     except Exception as e:
-        logger.error(f"Fehler bei der Suche: {str(e)}")
+        print(f"Fehler bei der Suche: {str(e)}")
         return jsonify({'error': str(e)}), 500
 
 def clean_dataframe(df):
@@ -329,12 +353,7 @@ def search_customers():
                 c.mobil,
                 c.email,
                 c.fachrichtung,
-                c.tag,
-                CASE 
-                    WHEN c.tag = 'medisoft' THEN 'MEDISOFT'
-                    WHEN c.tag = 'mediconsult' THEN 'MEDICONSULT'
-                    ELSE c.tag
-                END as formatted_tag
+                c.tag
             FROM customers c
             WHERE 1=1
         '''
@@ -420,7 +439,7 @@ def search_customers():
                 'mobil': row[6],
                 'email': row[7],
                 'fachrichtung': row[8],
-                'tag': row[10]  # Verwende den formatierten Tag
+                'tag': row[9]  # Tag-Feld hinzugefügt
             }
             formatted_results.append(customer)
             

static/css/styles.css

@@ -271,3 +271,23 @@ body {
 .tag-mediconsult {
     background-color: #ff9800;
 }
+
+/* Adress-Links */
+.address-text {
+    margin-right: 5px;
+}
+
+.address-link, .route-link {
+    color: #666;
+    text-decoration: none;
+    margin-left: 5px;
+    transition: color 0.2s;
+}
+
+.address-link:hover, .route-link:hover {
+    color: #0d6efd;
+}
+
+.location-pin, .route-pin {
+    font-size: 1.1em;
+} 

templates/index.html

@@ -110,7 +110,7 @@
     <footer class="footer">
         <div class="footer-content">
             Proudly made with ❤️ and 🍺 by <a href="https://www.medisoftware.de" target="_blank" class="footer-link">medisoftware</a>
-            <div style="font-size: 0.8em;">Version: v1.2.6</div>
+            <div style="font-size: 0.8em;">Version: v1.2.9</div>
         </div>
     </footer>
 
@@ -119,7 +119,7 @@
         let lastResults = [];
 
         function createPhoneLink(phone) {
-            if (!phone) return 'N/A';
+            if (!phone) return '';
             
             const clientIP = '{{ request.headers.get("X-Forwarded-For", request.remote_addr) }}';
             const allowedIPRanges = '{{ allowed_ip_ranges }}'.split(',');
@@ -143,7 +143,7 @@
         }
 
         function createEmailLink(email) {
-            if (!email) return 'N/A';
+            if (!email) return '';
             return `<a href="mailto:${email}" class="email-link">${email}</a>`;
         }
 
@@ -154,7 +154,7 @@
         }
 
         function createAddressLink(street, plz, city) {
-            if (!street || !plz || !city) return 'N/A';
+            if (!street || !plz || !city) return '';
             const address = `${street}, ${plz} ${city}`;
             const searchQuery = encodeURIComponent(address);
             const routeQuery = encodeURIComponent(address);
@@ -162,11 +162,11 @@
             return `<span class="address-text">${address}</span>
                     <a href="https://www.google.com/maps/search/?api=1&query=${searchQuery}" 
                     class="address-link" target="_blank" rel="noopener noreferrer">
-                    <i class="fa-solid fa-location-pin location-pin"></i>
+                    <i class="fa-solid fa-location-dot location-pin"></i>
                    </a>
                    <a href="https://www.google.com/maps/dir/?api=1&destination=${routeQuery}" 
                    class="route-link" target="_blank" rel="noopener noreferrer">
-                   <i class="fa-solid fa-car route-pin"></i>
+                   <i class="fa-solid fa-route route-pin"></i>
                    </a>`;
         }
 
@@ -243,7 +243,7 @@
             const resultsDiv = document.getElementById('results');
             const resultCount = document.getElementById('resultCount');
             
-            if (results.length === 0) {
+            if (!results || results.length === 0) {
                 resultsDiv.innerHTML = '<p>Keine Ergebnisse gefunden.</p>';
                 resultCount.textContent = '0 Ergebnisse';
                 return;
@@ -251,22 +251,39 @@
             
             resultCount.textContent = `${results.length} Ergebnisse`;
             
-            const resultsList = results.map(customer => `
+            const resultsList = results.map(customer => {
+                // Hilfsfunktion zum Erstellen von Feldern nur wenn sie Werte haben
+                const createFieldIfValue = (label, value, formatter = (v) => v) => {
+                    if (!value || value === 'N/A' || value === 'n/a' || value === 'N/a' || (typeof value === 'string' && value.trim() === '')) return '';
+                    const formattedValue = formatter(value);
+                    return `<p><strong>${label}:</strong> ${formattedValue}</p>`;
+                };
+
+                return `
                 <div class="card mb-3">
                     <div class="card-body">
                         <div class="d-flex justify-content-between align-items-start">
                             <h5 class="card-title">${customer.name}</h5>
-                            <button class="btn btn-sm btn-outline-primary" onclick="copyCustomerLink('${customer.nummer}')">
-                                <i class="fas fa-share-alt"></i> Teilen
-                            </button>
+                            <div class="d-flex align-items-center gap-2">
+                                <span class="badge ${customer.tag === 'medisoft' ? 'bg-primary' : 'bg-warning text-dark'}">${customer.tag.toUpperCase()}</span>
+                                <button class="btn btn-sm btn-outline-primary" onclick="copyCustomerLink('${customer.nummer}')">
+                                    <i class="fas fa-share-alt"></i> Teilen
+                                </button>
+                            </div>
                         </div>
                         <div class="card-text">
-                            <p><strong>Nummer:</strong> ${customer.nummer}</p>
-                            <p><strong>Adresse:</strong> ${customer.strasse}, ${customer.plz} ${customer.ort}</p>
-                            <p><strong>Telefon:</strong> ${createPhoneLink(customer.telefon)}</p>
-                            <p><strong>Mobil:</strong> ${createPhoneLink(customer.mobil)}</p>
-                            <p><strong>E-Mail:</strong> ${createEmailLink(customer.email)}</p>
-                            <p><strong>Fachrichtung:</strong> ${customer.fachrichtung}</p>
+                            ${createFieldIfValue('Nummer', customer.nummer)}
+                            ${createFieldIfValue('Adresse', (customer.strasse && customer.plz && customer.ort) ? true : false, 
+                                () => createAddressLink(customer.strasse, customer.plz, customer.ort))}
+                            ${createFieldIfValue('Telefon', customer.telefon, createPhoneLink)}
+                            ${createFieldIfValue('Mobil', customer.mobil, createPhoneLink)}
+                            ${createFieldIfValue('Handy', customer.handy, createPhoneLink)}
+                            ${createFieldIfValue('Telefon Firma', customer.tele_firma, createPhoneLink)}
+                            ${createFieldIfValue('E-Mail', customer.email, createEmailLink)}
+                            ${createFieldIfValue('Fachrichtung', customer.fachrichtung)}
+                            ${createFieldIfValue('Kontakt 1', customer.kontakt1, createPhoneLink)}
+                            ${createFieldIfValue('Kontakt 2', customer.kontakt2, createPhoneLink)}
+                            ${createFieldIfValue('Kontakt 3', customer.kontakt3, createPhoneLink)}
                             ${customer.tags && customer.tags.length > 0 ? `
                                 <p><strong>Tags:</strong> 
                                     ${customer.tags.map(tag => `<span class="badge bg-primary me-1">${tag}</span>`).join('')}
@@ -275,7 +292,7 @@
                         </div>
                     </div>
                 </div>
-            `).join('');
+            `}).join('');
             
             resultsDiv.innerHTML = resultsList;
         }
@@ -306,12 +323,18 @@
 
             // Führe die Suche durch
             fetch('/search?' + params.toString())
-                .then(response => response.json())
+                .then(response => {
+                    if (!response.ok) {
+                        throw new Error('Netzwerk-Antwort war nicht ok');
+                    }
+                    return response.json();
+                })
                 .then(data => {
                     // Verstecke das Lade-Icon
                     document.getElementById('loading').style.display = 'none';
                     
                     if (data.error) {
+                        console.error('Fehler bei der Suche:', data.error);
                         return;
                     }
                     
@@ -321,6 +344,8 @@
                 })
                 .catch(error => {
                     document.getElementById('loading').style.display = 'none';
+                    console.error('Fehler bei der Suche:', error);
+                    document.getElementById('results').innerHTML = '<p>Ein Fehler ist aufgetreten. Bitte versuchen Sie es später erneut.</p>';
                 });
         }