Project

General

Profile

Actions
Copy link

Bug #1638

closed

When editing a user and adding/editing the email address, trailing whitespace causes an input validation error.

Added by Brian Gupta about 13 years ago. Updated almost 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Ohad Levy
Category:
-
Target version:
1.0
Difficulty:
trivial
Triaged:
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

We can probably be a bit smarter about this and ignore leading and trailing whitespace.

Actions
Copy link
 #1

Updated by Ohad Levy about 13 years ago

sure, this sounds like a simple before_save callback, with sometihng like

before_save :normalize_email

private

def normalize_email
  self.email.strip! # or whatever is needed to fix the string
end
Actions
Copy link
 #2

Updated by Ohad Levy about 13 years ago

also worth mentioning that it would be great to have a simple unit test which compares the two scenarios (see test/unit/user.rb)

Actions
Copy link
 #3

Updated by Ohad Levy almost 13 years ago

  • Difficulty set to trivial
Actions
Copy link
 #4

Updated by Ohad Levy almost 13 years ago

  • Assignee set to Ohad Levy
  • Target version set to 1.0
Actions
Copy link
 #5

Updated by Ohad Levy almost 13 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100
Actions
Copy link

Also available in: Atom PDF