diff --git a/Gemfile b/Gemfile index c06c5d7..9a8aaa6 100755 --- a/Gemfile +++ b/Gemfile @@ -40,7 +40,7 @@ gem "kamal", require: false gem "thruster", require: false # Use Active Storage variants [https://guides.rubyonrails.org/active_storage_overview.html#transforming-images] -# gem "image_processing", "~> 1.2" +gem "image_processing", "~> 1.2" group :development, :test do # See https://guides.rubyonrails.org/debugging_rails_applications.html#debugging-with-the-debug-gem diff --git a/Gemfile.lock b/Gemfile.lock index 12296b9..8525c29 100755 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -122,6 +122,13 @@ GEM erubi (1.13.1) et-orbi (1.3.0) tzinfo + ffi (1.17.2-aarch64-linux-gnu) + ffi (1.17.2-aarch64-linux-musl) + ffi (1.17.2-arm-linux-gnu) + ffi (1.17.2-arm-linux-musl) + ffi (1.17.2-x86_64-darwin) + ffi (1.17.2-x86_64-linux-gnu) + ffi (1.17.2-x86_64-linux-musl) fugit (1.11.2) et-orbi (~> 1, >= 1.2.11) raabro (~> 1.4) @@ -129,6 +136,9 @@ GEM activesupport (>= 6.1) i18n (1.14.7) concurrent-ruby (~> 1.0) + image_processing (1.14.0) + mini_magick (>= 4.9.5, < 6) + ruby-vips (>= 2.0.17, < 3) io-console (0.8.1) irb (1.15.2) pp (>= 0.6.0) @@ -177,6 +187,8 @@ GEM net-smtp marcel (1.0.4) matrix (0.4.3) + mini_magick (5.3.1) + logger mini_mime (1.1.5) minitest (5.25.5) minitest-reporters (1.7.1) @@ -333,6 +345,9 @@ GEM rubocop-performance (>= 1.24) rubocop-rails (>= 2.30) ruby-progressbar (1.13.0) + ruby-vips (2.2.5) + ffi (~> 1.12) + logger ruby2_keywords (0.0.5) rubyzip (3.0.2) securerandom (0.4.1) @@ -429,6 +444,7 @@ DEPENDENCIES debug devise (~> 4.9) dotenv-rails + image_processing (~> 1.2) jbuilder jsbundling-rails kamal diff --git a/app/controllers/promoter/events_controller.rb b/app/controllers/promoter/events_controller.rb index e86b8bb..341ccff 100644 --- a/app/controllers/promoter/events_controller.rb +++ b/app/controllers/promoter/events_controller.rb @@ -134,10 +134,18 @@ class Promoter::EventsController < ApplicationController end def event_params - params.require(:event).permit( - :name, :slug, :description, :image, - :venue_name, :venue_address, :latitude, :longitude, - :start_time, :end_time, :featured, :allow_booking_during_event - ) + if action_name == 'create' + params.require(:event).permit( + :name, :slug, :description, :image, + :venue_name, :venue_address, :latitude, :longitude, + :start_time, :end_time, :featured, :allow_booking_during_event + ) + else + params.require(:event).permit( + :name, :description, :image, + :venue_name, :venue_address, :latitude, :longitude, + :start_time, :end_time, :featured, :allow_booking_during_event + ) + end end end diff --git a/app/javascript/controllers/event_form_controller.js b/app/javascript/controllers/event_form_controller.js index 0b7f431..803a1db 100644 --- a/app/javascript/controllers/event_form_controller.js +++ b/app/javascript/controllers/event_form_controller.js @@ -1,8 +1,11 @@ import { Controller } from "@hotwired/stimulus" import slug from 'slug' +// Configure slug to match Rails parameterize behavior +slug.defaults.mode = 'rfc3986' + export default class extends Controller { - static targets = ["name", "slug", "latitude", "longitude", "address", "mapLinksContainer", "geocodingSpinner", "getCurrentLocationBtn", "getCurrentLocationIcon", "getCurrentLocationText", "previewLocationBtn", "previewLocationIcon", "previewLocationText", "messagesContainer"] + static targets = ["name", "slug", "latitude", "longitude", "address", "mapLinksContainer", "geocodingSpinner", "getCurrentLocationBtn", "getCurrentLocationIcon", "getCurrentLocationText", "previewLocationBtn", "previewLocationIcon", "previewLocationText", "messagesContainer", "venueName"] static values = { geocodeDelay: { type: Number, default: 1500 } // Delay before auto-geocoding } @@ -27,15 +30,65 @@ export default class extends Controller { } } - // Generate slug from name + // Generate slug from name, venue name, and city for better SEO generateSlug() { const name = this.nameTarget.value + const venueName = this.hasVenueNameTarget ? this.venueNameTarget.value : "" + const address = this.hasAddressTarget ? this.addressTarget.value : "" - this.slugTarget.value = slug(name) + // Extract city from address + const city = this.extractCity(address) + + // Build SEO-friendly slug: name-venue-city + let slugParts = [] + + if (name) slugParts.push(name) + if (venueName) slugParts.push(venueName) + if (city) slugParts.push(city) + + let slugValue = slugParts.join('-') + + // If no slug parts, generate a fallback slug + if (!slugValue) { + slugValue = `event-${Date.now()}` + } + + // Generate slug with proper character handling (matches Rails parameterize) + this.slugTarget.value = slug(slugValue, { lower: true }) + } + + // Extract city from address + extractCity(address) { + if (!address) return "" + + // Look for French postal code pattern (5 digits) + city + const match = address.match(/(\d{5})\s+([^,]+)/) + if (match) { + return match[2].trim() + } + + // Fallback: extract last part after comma (assume it's city) + const parts = address.split(',') + if (parts.length > 1) { + return parts[parts.length - 1].trim() + } + + // Another fallback: look for common French city indicators + const cityIndicators = ["Paris", "Lyon", "Marseille", "Toulouse", "Nice", "Nantes", "Strasbourg", "Montpellier", "Bordeaux", "Lille"] + for (const city of cityIndicators) { + if (address.toLowerCase().includes(city.toLowerCase())) { + return city + } + } + + return "" } // Handle address changes with debounced geocoding addressChanged() { + // Regenerate slug when address changes + this.generateSlug() + // Clear any existing timeout if (this.geocodeTimeout) { clearTimeout(this.geocodeTimeout) @@ -68,6 +121,11 @@ export default class extends Controller { }, this.geocodeDelayValue) } + // Handle venue name changes to regenerate slug + venueNameChanged() { + this.generateSlug() + } + // Get user's current location and reverse geocode to address async getCurrentLocation() { if (!navigator.geolocation) { @@ -516,14 +574,16 @@ export default class extends Controller { showLocationSuccess(message) { this.hideAllLocationMessages() this.showMessage("location-success", message, "success") - setTimeout(() => this.hideMessage("location-success"), 4000) + // Keep notification visible indefinitely + // setTimeout(() => this.hideMessage("location-success"), 4000) } // Show error message showLocationError(message) { this.hideAllLocationMessages() this.showMessage("location-error", message, "error") - setTimeout(() => this.hideMessage("location-error"), 6000) + // Keep notification visible indefinitely + // setTimeout(() => this.hideMessage("location-error"), 6000) } // Show geocoding warning (less intrusive than error) @@ -531,7 +591,8 @@ export default class extends Controller { this.hideMessage("geocoding-warning") const message = "Les coordonnées n'ont pas pu être déterminées automatiquement. L'événement utilisera une localisation approximative." this.showMessage("geocoding-warning", message, "warning") - setTimeout(() => this.hideMessage("geocoding-warning"), 8000) + // Keep notification visible indefinitely + // setTimeout(() => this.hideMessage("geocoding-warning"), 8000) } // Show info about approximate location @@ -539,7 +600,8 @@ export default class extends Controller { this.hideMessage("approximate-location-info") const message = `Localisation approximative trouvée: ${foundLocation}` this.showMessage("approximate-location-info", message, "info") - setTimeout(() => this.hideMessage("approximate-location-info"), 6000) + // Keep notification visible indefinitely + // setTimeout(() => this.hideMessage("approximate-location-info"), 6000) } // Show geocoding success with location details @@ -547,7 +609,8 @@ export default class extends Controller { this.hideMessage("geocoding-success") const message = `${title}
${location}` this.showMessage("geocoding-success", message, "success") - setTimeout(() => this.hideMessage("geocoding-success"), 5000) + // Keep notification visible indefinitely + // setTimeout(() => this.hideMessage("geocoding-success"), 5000) } // Show geocoding progress with strategy info diff --git a/app/models/event.rb b/app/models/event.rb index d4be2c9..62e9e97 100755 --- a/app/models/event.rb +++ b/app/models/event.rb @@ -26,6 +26,7 @@ class Event < ApplicationRecord # === Callbacks === + before_validation :generate_slug, if: :should_generate_slug? before_validation :geocode_address, if: :should_geocode_address? before_update :handle_image_replacement, if: :image_attached? @@ -62,6 +63,71 @@ class Event < ApplicationRecord # === Instance Methods === + # Generate SEO-friendly slug from name, venue name, and city + def generate_slug + return if name.blank? && venue_name.blank? && venue_address.blank? + + # Extract city from venue address + city = extract_city_from_address(venue_address) + + # Build slug parts + slug_parts = [] + slug_parts << name if name.present? + slug_parts << venue_name if venue_name.present? + slug_parts << city if city.present? + + # Generate slug using Rails' parameterize + slug_value = slug_parts.join('-').parameterize + + # Ensure minimum length + if slug_value.length < 3 + slug_value = "event-#{Time.current.to_i}".parameterize + end + + # Make sure slug is unique + base_slug = slug_value + counter = 1 + while Event.where.not(id: id).where(slug: slug_value).exists? + slug_value = "#{base_slug}-#{counter}".parameterize + counter += 1 + end + + self.slug = slug_value + end + + # Check if slug should be generated + def should_generate_slug? + # Generate slug if it's blank or if it's a new record + slug.blank? || new_record? + end + + # Extract city from address + def extract_city_from_address(address) + return "" if address.blank? + + # Look for French postal code pattern (5 digits) + city + match = address.match(/(\d{5})\s+([^,]+)/) + if match + return match[2].strip + end + + # Fallback: extract last part after comma (assume it's city) + parts = address.split(',') + if parts.length > 1 + return parts[parts.length - 1].strip + end + + # Another fallback: look for common French city indicators + city_indicators = ["Paris", "Lyon", "Marseille", "Toulouse", "Nice", "Nantes", "Strasbourg", "Montpellier", "Bordeaux", "Lille"] + for city in city_indicators + if address.downcase.include?(city.downcase) + return city + end + end + + "" + end + # Get image URL prioritizing old image field if it exists def display_image_url # First check if old image field exists and has a value diff --git a/app/views/promoter/events/edit.html.erb b/app/views/promoter/events/edit.html.erb index 90b05c0..cc114eb 100644 --- a/app/views/promoter/events/edit.html.erb +++ b/app/views/promoter/events/edit.html.erb @@ -41,24 +41,9 @@

Informations générales

-
-
- <%= form.label :name, "Nom de l'événement", class: "block text-sm font-medium text-gray-700 mb-2" %> - <%= form.text_field :name, class: "w-full px-4 py-2 border border-gray-300 rounded-lg focus:ring-2 focus:ring-purple-500 focus:border-transparent", placeholder: "Ex: Soirée d'ouverture", data: { "event-form-target": "name", action: "input->event-form#generateSlug" } %> -
- -
- <%= form.label :slug, "Slug (URL)", class: "block text-sm font-medium text-gray-700 mb-2" %> - <%= form.text_field :slug, class: "w-full px-4 py-2 border border-gray-300 rounded-lg focus:ring-2 focus:ring-purple-500 focus:border-transparent", placeholder: "soiree-ouverture", data: { "event-form-target": "slug" } %> -

- <% if @event.published? %> - - Attention: Modifier le slug d'un événement publié peut casser les liens existants. - <% else %> - Utilisé dans l'URL de l'événement - <% end %> -

-
+
+ <%= form.label :name, "Nom de l'événement", class: "block text-sm font-medium text-gray-700 mb-2" %> + <%= form.text_field :name, class: "w-full px-4 py-2 border border-gray-300 rounded-lg focus:ring-2 focus:ring-purple-500 focus:border-transparent", placeholder: "Ex: Soirée d'ouverture" %>
diff --git a/app/views/promoter/events/new.html.erb b/app/views/promoter/events/new.html.erb index 84bb73b..ab95d42 100644 --- a/app/views/promoter/events/new.html.erb +++ b/app/views/promoter/events/new.html.erb @@ -41,17 +41,14 @@

Informations générales

-
+
<%= form.label :name, "Nom de l'événement", class: "block text-sm font-medium text-gray-700 mb-2" %> <%= form.text_field :name, class: "w-full px-4 py-2 border border-gray-300 rounded-lg focus:ring-2 focus:ring-purple-500 focus:border-transparent", placeholder: "Ex: Soirée d'ouverture", data: { "event-form-target": "name", action: "input->event-form#generateSlug" } %>
-
- <%= form.label :slug, "Slug (URL)", class: "block text-sm font-medium text-gray-700 mb-2" %> - <%= form.text_field :slug, class: "w-full px-4 py-2 border border-gray-300 rounded-lg focus:ring-2 focus:ring-purple-500 focus:border-transparent", placeholder: "soiree-ouverture", data: { "event-form-target": "slug" } %> -

Utilisé dans l'URL de l'événement

-
+ + <%= form.hidden_field :slug, data: { "event-form-target": "slug" } %>
@@ -122,7 +119,7 @@
<%= form.label :venue_name, "Nom du lieu", class: "block text-sm font-medium text-gray-700 mb-2" %> - <%= form.text_field :venue_name, class: "w-full px-4 py-2 border border-gray-300 rounded-lg focus:ring-2 focus:ring-purple-500 focus:border-transparent", placeholder: "Ex: Le Grand Rex" %> + <%= form.text_field :venue_name, class: "w-full px-4 py-2 border border-gray-300 rounded-lg focus:ring-2 focus:ring-purple-500 focus:border-transparent", placeholder: "Ex: Le Grand Rex", data: { "event-form-target": "venueName", action: "input->event-form#venueNameChanged" } %>