Merge pull request #173 from denis1011101/fix/user-city-from-catalog

fix: город в профиле сохраняем только выбором из справочника

Author: denis1011101

app/controllers/users_controller.rb

@@ -2,6 +2,7 @@ class UsersController < ApplicationController
   include LocationFilters
 
   CITY_SEARCH_DEFAULT_LIMIT = 5
+  CITY_SEARCH_MAX_LIMIT = 20
 
   before_action :authenticate_user!
 
@@ -38,13 +39,13 @@ def update
     court_preferences_submitted = user_params.key?(:court_preferences_mode) || user_params.key?(:favorite_court_ids)
     court_preferences_mode = user_params[:court_preferences_mode].presence || default_court_preferences_mode(@user)
 
-    query = user_attrs["city_name"].to_s.strip
-    if query.present?
-      coords_regex = /\A\s*-?\d+(\.\d+)?\s*,\s*-?\d+(\.\d+)?\s*\z/
-      # for plain names: transliterate immediately so DB shows e.g. "Kurgan"
-      unless query =~ coords_regex
-        user_attrs["city_name"] = translit_str(query)
-      end
+    # Город принимаем только выбором из подсказок: произвольный текст в поле
+    # раньше уезжал в city_name как есть и ломал матчинг с courts.city_name.
+    user_attrs.delete("city_name")
+    selected_city = City.find_by(id: params[:selected_city_id]) if params[:selected_city_id].present?
+    if selected_city
+      user_attrs["city_name"] = selected_city.canonical_name
+      user_attrs["timezone"] = selected_city.rails_timezone if selected_city.rails_timezone.present?
     end
 
     if court_preferences_submitted
@@ -63,12 +64,6 @@ def update
     Rails.logger.info "[UsersController#update] saving user_attrs=#{user_attrs.inspect}"
 
     if @user.update(user_attrs)
-      # enqueue background job to resolve timezone asynchronously (by coords or by name)
-      if query.present?
-        ResolveUserCityJob.perform_later(@user.id, query)
-        Rails.logger.info "[UsersController#update] enqueued ResolveUserCityJob for user_id=#{@user.id} query=#{query.inspect}"
-      end
-
       respond_to do |format|
         format.html { redirect_to update_section_path(section), notice: "Account updated" }
         format.json { render json: { success: true, city_name: @user.city_name, timezone: @user.timezone } }
@@ -95,6 +90,19 @@ def dismiss_onboarding
     redirect_back fallback_location: root_path
   end
 
+  # Подсказки для поля города: отдаём только то, что есть в справочнике, —
+  # сохранить можно лишь выбранную строку.
+  def city_search
+    # Потолок обязателен: в SQLite отрицательный LIMIT снимает ограничение
+    # совсем, и запрос вытащил бы весь справочник городов целиком.
+    limit = params[:limit].present? ? params[:limit].to_i.clamp(1, CITY_SEARCH_MAX_LIMIT) : CITY_SEARCH_DEFAULT_LIMIT
+    cities = Cities::SearchService.new(query: params[:q], limit: limit).call
+
+    render json: cities.map { |city|
+      { id: city.id, name: city.canonical_name, hint: [ city.country_code, city.timezone ].compact_blank.join(" · ") }
+    }
+  end
+
   def clear_city
     @user = current_user
     @user.update(timezone: nil, city_name: nil)
@@ -177,10 +185,6 @@ def destroy
 
   private
 
-  def translit_str(s)
-    Russian.translit(s.to_s)
-  end
-
   def user_update_params
     params.require(:user).permit(
       :name,
@@ -205,15 +209,15 @@ def prepare_notifications_form_state
     @registration_token = @user.ensure_telegram_registration_token!
   end
 
+  # Раньше здесь по таймзоне подбирался «какой-нибудь» город из справочника и
+  # подставлялся в поле города. Таймзона города не определяет — в
+  # Asia/Yekaterinburg лежит и Челябинск, и Тюмень, — поэтому не подставляем
+  # ничего: пустое поле честнее угаданного.
   def prepare_profile_form_state
-    @limit = params[:limit].to_i.nonzero? || CITY_SEARCH_DEFAULT_LIMIT
-
-    if params[:selected_city_name].present?
-      @selected_city_name = params[:selected_city_name]
-    else
-      c = City.find_by(timezone: @user.timezone)
-      @selected_city_name = "#{c.name}, #{c.country_code} — #{c.timezone}" if c.present?
-    end
+    # После ошибки валидации форму рисуем заново, и выбранный город надо вернуть
+    # в hidden-поле: в видимом поле он уже стоит, а без id следующая отправка
+    # город не изменит.
+    @selected_city_id = params[:selected_city_id].presence
   end
 
   def prepare_court_preferences_state

app/javascript/controllers/city_picker_controller.js

@@ -0,0 +1,132 @@
+import { Controller } from "@hotwired/stimulus"
+
+// Поле города в профиле. Сохранить можно только город из справочника, поэтому
+// видимый текст сам по себе ничего не значит: город несёт hidden-поле с id, и
+// любое ручное редактирование текста этот id сбрасывает — иначе к новому тексту
+// прицепился бы id старого выбора.
+export default class extends Controller {
+  static targets = ["input", "cityId", "results"]
+  static values = {
+    url: String,
+    searching: String,
+    noMatches: String,
+    error: String
+  }
+
+  connect() {
+    this.searchTimeout = null
+    this.requestId = 0
+    this.hideTimeout = null
+    this.onDocumentClick = (event) => {
+      if (!this.element.contains(event.target)) this.hide()
+    }
+    document.addEventListener("click", this.onDocumentClick)
+  }
+
+  disconnect() {
+    clearTimeout(this.searchTimeout)
+    clearTimeout(this.hideTimeout)
+    document.removeEventListener("click", this.onDocumentClick)
+  }
+
+  // Правка текста отменяет выбор: иначе к новому названию прицепился бы id
+  // прошлого города.
+  search() {
+    this.cityIdTarget.value = ""
+    this.startRequest()
+
+    const query = this.inputTarget.value.trim()
+    if (query.length < 2) {
+      this.hide()
+      return
+    }
+
+    this.searchTimeout = setTimeout(() => this.fetchCities(query), 300)
+  }
+
+  // Фокус в поле лишь возвращает список: выбранный город остаётся выбранным.
+  reopen() {
+    if (this.cityIdTarget.value) return
+
+    this.search()
+  }
+
+  // Поколение запроса растёт на каждое изменение запроса, включая уход в
+  // слишком короткую строку, — иначе подвисший ответ снова открыл бы список
+  // с городами, которых в поле уже нет.
+  startRequest() {
+    clearTimeout(this.searchTimeout)
+    this.requestId = (this.requestId || 0) + 1
+    return this.requestId
+  }
+
+  async fetchCities(query) {
+    this.renderMessage(this.searchingValue)
+    const requestId = this.requestId
+
+    try {
+      const response = await fetch(`${this.urlValue}?q=${encodeURIComponent(query)}`, {
+        headers: { Accept: "application/json" }
+      })
+      if (!response.ok) throw new Error(response.status)
+      const cities = await response.json()
+      // Ответы могут прийти не в том порядке, в каком уходили запросы.
+      if (requestId !== this.requestId) return
+      this.renderCities(cities)
+    } catch (error) {
+      console.warn("City search failed:", error)
+      if (requestId === this.requestId) this.renderMessage(this.errorValue)
+    }
+  }
+
+  renderCities(cities) {
+    if (!cities.length) {
+      this.renderMessage(this.noMatchesValue)
+      return
+    }
+
+    this.resultsTarget.innerHTML = cities
+      .map(
+        (city) => `
+          <button type="button" data-action="click->city-picker#select"
+                  data-id="${city.id}" data-name="${escapeAttribute(city.name)}"
+                  class="block w-full border-b px-3 py-2 text-left text-sm last:border-b-0 hover:bg-gray-100 dark:border-white/10 dark:hover:bg-white/5">
+            <span class="font-medium">${escapeHtml(city.name)}</span>
+            <span class="ml-1 text-xs text-gray-500 dark:text-slate-400">${escapeHtml(city.hint)}</span>
+          </button>`
+      )
+      .join("")
+    this.show()
+  }
+
+  renderMessage(text) {
+    this.resultsTarget.innerHTML = `<div class="px-3 py-2 text-center text-sm text-gray-500 dark:text-slate-400">${escapeHtml(text)}</div>`
+    this.show()
+  }
+
+  select(event) {
+    const { id, name } = event.currentTarget.dataset
+    this.startRequest()
+    this.cityIdTarget.value = id
+    this.inputTarget.value = name
+    this.hide()
+  }
+
+  show() {
+    this.resultsTarget.classList.remove("hidden")
+  }
+
+  hide() {
+    this.resultsTarget.classList.add("hidden")
+  }
+}
+
+function escapeHtml(value) {
+  const div = document.createElement("div")
+  div.textContent = value == null ? "" : String(value)
+  return div.innerHTML
+}
+
+function escapeAttribute(value) {
+  return escapeHtml(value).replace(/"/g, "&quot;")
+}

app/jobs/resolve_user_city_job.rb

@@ -18,11 +18,8 @@ def perform(user_id, original_query)
 
     return unless city
 
-    new_name = city.respond_to?(:asciiname) ? city.asciiname.presence || translit_str(city.name) : translit_str(city.name)
-
-    # map IANA -> Rails display name
-    rails_zone = ActiveSupport::TimeZone.all.find { |z| z.tzinfo.name == city.timezone } rescue nil
-    tz_to_set = rails_zone ? rails_zone.name : (ActiveSupport::TimeZone[city.timezone].present? ? city.timezone : nil)
+    new_name = city.canonical_name
+    tz_to_set = city.rails_timezone
 
     # avoid clobbering if user changed city manually after save:
     expected_current = original_query =~ coords_regex ? original_query : translit_str(original_query)

app/models/city.rb

@@ -12,4 +12,20 @@ def self.country_codes_for(names)
       .group_by(&:first)
       .transform_values { |rows| rows.max_by { |(_, _, population)| population.to_i }[1] }
   end
+
+  # Каноническое имя города для профиля и матчинга с courts.city_name: берём
+  # name, а не asciiname — в кортах города записаны с диакритикой («Båstad»,
+  # «Acapulco de Juárez»), и asciiname с ними бы не совпал.
+  def canonical_name
+    name.presence || asciiname.presence
+  end
+
+  # GeoNames отдаёт IANA-зону, а часть кода ждёт отображаемое имя Rails.
+  # Возвращаем то, что Rails понимает, иначе — nil, чтобы не записать мусор.
+  def rails_timezone
+    zone = ActiveSupport::TimeZone.all.find { |z| z.tzinfo.name == timezone }
+    return zone.name if zone
+
+    timezone if ActiveSupport::TimeZone[timezone.to_s].present?
+  end
 end

app/models/user.rb

@@ -1,5 +1,5 @@
 class User < ApplicationRecord
-  before_validation :normalize_email, :titleize_city_name, :set_default_notification_channel
+  before_validation :normalize_email, :set_default_notification_channel
   before_validation :set_default_registration_source, on: :create
 
   has_one :player_statistic, dependent: :destroy
@@ -198,11 +198,6 @@ def normalize_email
     true  # явно возвращаем true
   end
 
-  def titleize_city_name
-    self.city_name = city_name.to_s.titleize.presence
-    true  # <-- FIX: явно возвращаем true, чтобы не прерывать save
-  end
-
   def set_default_registration_source
     self.registration_source = "email" if registration_source.blank?
     true  # явно возвращаем true

app/views/users/_city_results.html.erb

@@ -32,10 +32,34 @@
     <p class="text-xs text-gray-500 dark:text-slate-400 mt-1"><%= t("users.profile.coach_hint") %></p>
   </div>
 
-  <div>
+  <%# Город сохраняется только выбором из подсказок: hidden-поле несёт id из
+      справочника, а видимый текст — лишь то, что человек читает. %>
+  <div data-controller="city-picker"
+       data-city-picker-url-value="<%= city_search_account_path %>"
+       data-city-picker-searching-value="<%= t("users.profile.city_searching") %>"
+       data-city-picker-no-matches-value="<%= t("users.profile.no_city_matches") %>"
+       data-city-picker-error-value="<%= t("users.profile.city_search_error") %>">
     <%= f.label :city_name, t("users.profile.city"), class: "block text-sm font-medium text-gray-700 dark:text-slate-300" %>
-    <%= f.text_field :city_name, value: (params[:city_query].presence || @user.city_name || @selected_city_name), placeholder: t("users.profile.city_placeholder"), class: "mt-1 block w-full rounded border px-3 py-2 dark:border-white/15 dark:bg-slate-700 dark:text-slate-100" %>
+    <%= hidden_field_tag :selected_city_id, @selected_city_id, data: { city_picker_target: "cityId" } %>
+    <div class="relative">
+      <%# На фокусе только открываем подсказки: сбрасывать выбранный город
+          нельзя — возврат в поле без правки текста молча отменял бы выбор. %>
+      <%= f.text_field :city_name, value: @user.city_name,
+            placeholder: t("users.profile.city_placeholder"),
+            autocomplete: "off",
+            data: { city_picker_target: "input", action: "input->city-picker#search focus->city-picker#reopen" },
+            class: "mt-1 block w-full rounded border px-3 py-2 dark:border-white/15 dark:bg-slate-700 dark:text-slate-100" %>
+      <div data-city-picker-target="results"
+           class="hidden absolute z-10 mt-1 w-full overflow-hidden rounded border bg-white shadow dark:border-white/15 dark:bg-slate-700"></div>
+    </div>
     <p class="text-sm text-gray-600 dark:text-slate-400 mt-2"><%= t("users.profile.city_hint") %></p>
+    <% if @user.city_name.present? %>
+      <%# Именно ссылка, а не button_to: поле города живёт внутри формы профиля,
+          а форма в форме — невалидная разметка. %>
+      <%= link_to t("users.profile.city_clear"), clear_city_account_path,
+            data: { turbo_method: :post },
+            class: "mt-2 inline-block text-sm text-indigo-600 hover:underline dark:text-indigo-400" %>
+    <% end %>
   </div>
 
   <div>

app/views/users/_profile_form.html.erb

@@ -389,9 +389,12 @@ en:
       coach_hint: Tick if you coach — this helps others find and hire coaches in the
         future.
       city: City
-      city_placeholder: e.g. Moskva
-      city_hint: Enter a place. The system will transliterate names and try to detect
-        timezone automatically on save. Press "Save Changes".
+      city_placeholder: e.g. Moscow
+      city_hint: Start typing and pick a city from the list — only a picked city is
+        saved. The timezone is filled in for you.
+      city_searching: Searching…
+      city_search_error: Could not load the city list.
+      city_clear: Clear city
       about_me: About me
       about_me_placeholder: Tell others a bit about yourself…
       no_city_matches: No matches.

config/locales/web.en.yml

@@ -393,8 +393,11 @@ es:
       coach_hint: 'Márcalo si entrenas: ayuda a que otros encuentren y contraten entrenadores.'
       city: Ciudad
       city_placeholder: p. ej. Madrid
-      city_hint: Escribe un lugar. Al guardar, el sistema transliterará el nombre
-        e intentará detectar la zona horaria. Pulsa «Guardar cambios».
+      city_hint: 'Empieza a escribir y elige una ciudad de la lista: solo se guarda
+        la ciudad elegida. La zona horaria se rellena sola.'
+      city_searching: Buscando…
+      city_search_error: No se pudo cargar la lista de ciudades.
+      city_clear: Quitar ciudad
       about_me: Sobre mí
       about_me_placeholder: Cuenta algo sobre ti…
       no_city_matches: Sin resultados.

config/locales/web.es.yml

@@ -394,8 +394,11 @@ ru:
         тренера.
       city: Город
       city_placeholder: например, Москва
-      city_hint: Укажите место. При сохранении система транслитерирует название и
-        попробует определить часовой пояс. Нажмите «Сохранить изменения».
+      city_hint: Начните вводить название и выберите город из списка — сохраняется
+        только выбранный вариант. Часовой пояс подставится сам.
+      city_searching: Ищем…
+      city_search_error: Не удалось загрузить список городов.
+      city_clear: Убрать город
       about_me: О себе
       about_me_placeholder: Расскажите немного о себе…
       no_city_matches: Ничего не найдено.