CF-510 randomize drupal user name to fix db insert issue

cultuurnet / culturefeed

Drupal module suite for building an event site based

http://www.culturefeed.be

Apache License 2.0

11 stars 17 forks source link

CF-510 randomize drupal user name to fix db insert issue #205

Closed lukdens closed 6 years ago

lukdens commented 6 years ago

Randomize the drupal user name for anonymous users to avoid it from getting too long to insert in db.

Example: A user name like "Anoniem_1_2_3_4_5_6_7_8_9_10_11_12_13_14_15_16_17_18_19_20_21" could not be inserted in a name field with a length of 60

zuuperman commented 6 years ago

@lukdens Waarom gebruik je niet het uid als username voor de anonieme gebruikers?

lukdens commented 6 years ago

@zuuperman dat is omdat de drupal username ook gebruikt wordt in de page title, URL,... en we daar dus livcer een Anoniem_1234 zien staan dan een semi-traceerbaar uid. Bovendien zal de private nick op uitid ook volgens die logica gegenereerd worden en aangezien deze nog kan verschijnen in activity feeds enzo, lijkt dat dus consistenter om ze telkens te prefixen met anoniem + 4 schijnbaar willekeurige karakters en niet de volledige uid te tonen.