search

ZeusWPI / Gandalf

You Shall Not Pass - An advanced e-ticket system for student clubs -
http://event.fkgent.be
MIT License
10 stars 7 forks source link

Wrong student number #313

Closed Thoge closed 10 years ago

Thoge commented 10 years ago

The student number filled in by the script is erroneous. The number filled in is 263547. My studenten number is 01002573. See also attached screenshot

studentennummer

TomNaessens commented 10 years ago

Debug information gives us: I, [2014-01-27T21:10:36.032646 #27671] INFO -- : LOX: mail: censored I, [2014-01-27T21:10:36.032932 #27671] INFO -- : LOX: givenname: censored I, [2014-01-27T21:10:36.033058 #27671] INFO -- : LOX: surname: censored I, [2014-01-27T21:10:36.033175 #27671] INFO -- : LOX: ugentStudentID: 263547 I, [2014-01-27T21:10:36.033290 #27671] INFO -- : LOX: uid: censored

Which would mean the problem is DICTs?!

maartenhbe commented 10 years ago

Mine is shown correctly. How about yours?

TomNaessens commented 10 years ago

So is mine. Weird stuff is happening here.

maartenhbe commented 10 years ago

Are there any updates on this one? Is it a devise or DICT issue? Too weird..

TomNaessens commented 10 years ago

I haven't had the time to investigate this furder. I'll do so in the following days.

2014-01-31, Maarten notifications@github.com:

Are there any updates on this one? Is it a devise or DICT issue? Too weird imho..

Reply to this email directly or view it on GitHub: https://github.com/ZeusWPI/Gandalf/issues/313#issuecomment-33808697

nudded commented 10 years ago

I asked for the ugentid as well so now we can debug more.

I do think we should hide the student number field once we have the information from CAS though

maartenhbe commented 10 years ago

Would you hide the field always or only when logged in via CAS?

If we would hide the field, we should also disable them to enter a different name. Else a member who isn't coming, can still register a ticket with member advantages for somebody else.

nudded commented 10 years ago

Yes it should show the fields but disable them indeed.

TomNaessens commented 10 years ago

This is the problem. It is used deep inside the rubycas-client dependency of devise_cas_authenticable where the extra attributes are parsed from YAML.

(Credits go to @Javache)

TomNaessens commented 10 years ago

The encoding can be set it seems as defined here.

TomNaessens commented 10 years ago

Fixed in #416