Project

General

Profile

Actions
Copy link

Task #1111

closed

Trim the whitespaces in reg form

Added by Marcel Poul over 6 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Marek Klement
Target version:
3.0.1
Start date:
05/28/2018
Due date:
% Done:

100%

Estimated time:
Owner:

Description

Sometimes users type in whitespace e.g. " " before or after the input in fields. The aim of this ticket is to trim the whitespaces in the reg. form.

Related issues

Related to IdStory Identity Manager - Task #1711: Trim or warn about leading and trailing whitespacesClosedOndrej Husník06/06/2019

Actions
Actions
Copy link
 #1

Updated by Ondřej Kopr over 6 years ago

  • Assignee set to Marcel Poul

This isn't bug registration module. Each project has own login generator. And you probably mean the generator on project A***. In default implementation registration module is only some example login generator that unite first name and last name. The main problem is in login generator on project.

Actions
Copy link
 #2

Updated by Marcel Poul over 6 years ago

I agree this is not a bug, this is a clear improvement. I think sooner the trim is used the better, cannot see any usage of whitespaces afterwords.

Actions
Copy link
 #3

Updated by Marcel Poul over 6 years ago

  • Assignee changed from Marcel Poul to Ondřej Kopr
Actions
Copy link
 #5

Updated by Ondřej Kopr over 6 years ago

  • Assignee changed from Ondřej Kopr to Peter Štrunc
Actions
Copy link
 #7

Updated by Radek Tomiška over 4 years ago

  • Related to Task #1711: Trim or warn about leading and trailing whitespaces added
Actions
Copy link
 #8

Updated by Radek Tomiška over 4 years ago

Maybe is already solved in product ticket #1711, if warning is sufficient?

Actions
Copy link
 #9

Updated by Ondřej Kopr over 4 years ago

  • Assignee changed from Peter Štrunc to Marek Klement
Actions
Copy link
 #10

Updated by Marek Klement over 4 years ago

  • Status changed from New to Needs feedback
  • Assignee changed from Marek Klement to Ondřej Kopr

I checked this with new version of product including warning on whitespaces. It looks good to me. This feature just warn about used whitespace before and after value - no trim used. Will it be ok with ok with you @kopro ?

If trim is needed, probably best solution is to create a processor which trims whitespaces at begining and at the end.

Actions
Copy link
 #11

Updated by Marek Klement over 4 years ago

  • Target version set to 3.0.1
Actions
Copy link
 #12

Updated by Ondřej Kopr over 4 years ago

  • Status changed from Needs feedback to Resolved
  • Assignee changed from Ondřej Kopr to Marek Klement
  • % Done changed from 0 to 100

The product behavior with warning is also good to me. Thank you for check. You can close the ticket.

Actions
Copy link
 #13

Updated by Marek Klement over 4 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF