adamvail / drupal_modules

Modules for Drupal
0 stars 0 forks source link

Normalize gym name #27

Closed adamvail closed 11 years ago

adamvail commented 11 years ago

We need to normalize gym names. One possible way around this is to only allow users to specify their gym if their coach is already registered.

ACAHNN commented 11 years ago

Is this pertinent since the LIKE command is case insensitive? All that will happen is their name will display oddly (i.e Madison and MaDiSoN will still affiliate together). We can standardize the output with the pretty_print function if desired.

adamvail commented 11 years ago

i think that it's fine for now. Why don't we normalize to all lower case when it is put in originally, and then pretty print it on output.

The harder part is when people put CF to stand for crossfit. So people would put CF Sanctify. Hopefully though when someone is adding a new one they see the examples of the other gyms.

Adam

On Sun, Apr 21, 2013 at 3:32 PM, ACAHNN notifications@github.com wrote:

Is this pertinent since the LIKE command is case insensitive? All that will happen is their name will display oddly (i.e Madison and MaDiSoN will still affiliate together). We can standardize the output with the pretty_print function if desired.

— Reply to this email directly or view it on GitHubhttps://github.com/adamvail/drupal_modules/issues/27#issuecomment-16745677 .

ACAHNN commented 11 years ago

all gyms normalize to lowercase as well as display names. validation logic for the gym affiliation code is now implemented.