Geen controle op dubbele gebruikersnamen

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Gebruik bij registratie een bestaande voornaam en achternaam e.g. hans then

What is the expected output? What do you see instead?
Dit zou een foutmelding of een waarschuwing moeten geven. In plaats daarvan 
worden de gegevens van de gebruiker eenvoudig overschreven. Overigens vindt ook 
geen controle plaats of het gebruikte e-mail adres of telefoonnummer wel uniek 
is.

Ik denk dat we de workflow van het registreren helemaal opnieuw moeten 
ontwerpen. Hier zal een two-phase registratie voor in de plaats moeten komen om 
te controleren of iemand wel echt de eigenaar is van een telefoonnummer of 
e-mail adres.

Please use labels and text to provide additional information.

Original issue reported on code.google.com by hans.t...@gmail.com on 11 Jun 2012 at 7:32

[GoogleCodeExporter]

Zie ook de nieuwe use case beschrijving.
https://docs.google.com/document/pub?id=1RPo1SUxiSrDM3CBczAhWm8skHyqulWKag6MU4dJ
ikOY

Original comment by hans.t...@gmail.com on 11 Jun 2012 at 6:53

[GoogleCodeExporter]

@Darius: als je wilt mag je dit aanpassen. Het is een stuk python code in de 
ClientForm in forms.py. Waar de aanpassingen moeten gebeuren spreekt redelijk 
voor zich. Je moet in de appengine API even controleren hoe je het beste kunt 
kijken of er al een Client met dezelfde username, telefoon of e-mail in de 
database bestaat. Zo ja, dan moet je een foutmelding op dat veld toevoegen.

Original comment by hans.t...@gmail.com on 17 Jun 2012 at 9:19

• Added labels: Priority-Critical
• Removed labels: Priority-High

[GoogleCodeExporter]

Original comment by dkruyth...@gmail.com on 18 Jun 2012 at 7:53

• Changed state: Fixed

[GoogleCodeExporter]

Ik heb de code bekeken. Wat me opviel: mobile en phone kunnen omgedraaid worden 
en moeten dus allebei bekeken worden. I.e. een controle of een mobiel nummer 
bestaat moet ook uitgevoerd worden in het phone veld. Verder is het efficienter 
om een keys_only query te doen. Het makkelijkst te gebruiken is dan iets als: 
models.Client.all(keys_only=False).filter("phone =", data["phone"])

Original comment by hans.t...@gmail.com on 18 Jun 2012 at 11:12

[GoogleCodeExporter]

Original comment by hans.t...@gmail.com on 3 Sep 2012 at 7:39

• Changed state: Verified