Bugfix: private claim cannot be named as such

psecio / jwt

A JWT (JSON Web Token) Encoder & Decoder

110 stars 13 forks source link

Bugfix: private claim cannot be named as such #3

Closed jschreuder closed 10 years ago

jschreuder commented 10 years ago

"Private" is a PHP keyword and is reserved, thus there cannot be a class with that name. For now I've renamed the class back to "Custom". Another solution would be to suffix all the types with "Claim", like "PrivateClaim" which would allow this. But I'll leave that up to you.

enygma commented 10 years ago

huh, interesting....I wonder why the updated tests passed then. weird. thanks for the change!