AO3-6587 Allow # as initial character in Ticket ID Field when editing a profile or pseud

app/models/concerns/justifiable.rb

@@ -5,16 +5,24 @@ module Justifiable
     attr_accessor :ticket_number
     attr_reader :ticket_url
 
+    before_validation :strip_octothorpe
     validates :ticket_number,
               presence: true,
-              numericality: { only_integer: true },
+              numericality: { only_integer: true,
+                              message: :numeric_with_optional_hash },
               if: :enabled?
 
     validate :ticket_number_exists_in_tracker, if: :enabled?
   end
 
   private
 
+  def strip_octothorpe
+    return if ticket_number.is_a?(Integer)
+
+    self.ticket_number = self.ticket_number.delete_prefix("#") unless self.ticket_number.nil?
+  end
+
   def enabled?
     # Only require a ticket if the record has been changed by an admin.
     User.current_user.is_a?(Admin) && changed?

app/views/pseuds/_pseuds_form.html.erb

@@ -62,8 +62,6 @@
     <dt><%= f.label :ticket_number, class: "required" %></dt>
     <dd>
       <%= f.text_field :ticket_number, class: "required" %>
-      <%= live_validation_for_field("pseud_ticket_number", numericality: true) %>
-      <p class="footnote"><%= t(".ticket_footnote") %></p>
     </dd>
   <% end %>
 

app/views/users/edit.html.erb

@@ -46,8 +46,6 @@
         <dt><%= p.label :ticket_number, class: "required" %></dt>
         <dd>
           <%= p.text_field :ticket_number, class: "required" %>
-          <%= live_validation_for_field("profile_attributes_ticket_number", numericality: true) %>
-          <p class="footnote"><%= t(".admin.ticket_number.numbers_only") %></p>
         </dd>
       <% end %>
 

config/locales/models/en.yml

@@ -94,6 +94,7 @@ en:
     errors:
       messages:
         forbidden: "%{value} is not allowed"
+        numeric_with_optional_hash: 'may begin with an # and otherwise contain only numbers.'
       models:
         abuse_report:
           attributes:

config/locales/views/en.yml

@@ -1594,7 +1594,6 @@ en:
       make_default: Make this name default
       name: Name
       submit: Submit
-      ticket_footnote: Numbers only.
   skins:
     confirm_delete:
       confirm_html: Are you sure you want to <strong><em>delete</em></strong> the skin "%{skin_title}"?
@@ -1744,9 +1743,6 @@ en:
       submit: Save
     edit:
       about_me: About Me
-      admin:
-        ticket_number:
-          numbers_only: Numbers only.
       browser_title: Edit Profile
       change_profile_landmark: Change Profile
       date_of_birth: Date of Birth

features/other_a/profile_edit.feature

@@ -42,7 +42,7 @@ Scenario: Change details as an admin
     And I fill in "About Me" with "is it merely thy habit, to talk to dolls?"
     And I fill in "Ticket ID" with "fine"
     And I press "Update"
-  Then I should see "Ticket ID is not a number"
+  Then I should see "may begin with an # and otherwise contain only numbers."
     And the field labeled "Ticket ID" should contain "fine"
   When I fill in "Ticket ID" with "480000"
     And I press "Update"

features/other_a/pseuds.feature

@@ -263,9 +263,18 @@ Scenario: Change details as an admin
     And I fill in "Description" with "I'd probably be removing text."
     And I fill in "Ticket ID" with "no 💜"
     And I press "Update"
-  Then I should see "Ticket ID is not a number"
+  Then I should see "may begin with an # and otherwise contain only numbers"
     And the field labeled "Ticket ID" should contain "no 💜"
+  When I fill in "Ticket ID" with "#4798454#331"
+    And I press "Update"
+  Then I should see "may begin with an # and otherwise contain only numbers"
+    And the field labeled "Ticket ID" should contain "4798454#331"
   When I fill in "Ticket ID" with "47"
+   And I press "Update"
+  Then I should see "Pseud was successfully updated."
+  When I go to someone's pseuds page
+   And I follow "Edit alt"
+  When I fill in "Ticket ID" with "#47"
     And I press "Update"
   Then I should see "Pseud was successfully updated."
   When I go to someone's pseuds page

spec/models/concerns/justifiable_spec.rb

@@ -26,7 +26,7 @@
 
       record.assign_attributes(attributes)
       expect(record).not_to be_valid
-      expect(record.errors[:ticket_number]).to contain_exactly("can't be blank", "is not a number")
+      expect(record.errors[:ticket_number]).to contain_exactly("can't be blank", "may begin with an # and otherwise contain only numbers.")
       expect(record.ticket_url).to be_nil
     end