lamernews icon indicating copy to clipboard operation
lamernews copied to clipboard

Wrong user account displayed on front page

Open mhdingler opened this issue 14 years ago • 1 comments

Just went to the lamernews front page and the wrong user profile was shown in the right hand corner.

Screenshot: http://i.imgur.com/s5R9R.png

My user name: mhd Wrong user: deivinsontejeda

Back to normal on reload. Couldn't reproduce error after that.

System: Firefox 7.0.1 (OS X 10.7.2) w/ AdBlock and Ghostery add-ons

mhdingler avatar Oct 28 '11 16:10 mhdingler

Probably due to the fact that I use a global for $user ?

Apparently in Sinatra globals are shared among the whole environment, will switch to Sinatra properties soon to fix that. Thanks for reporting.

EDIT: I don't think properties are a good fix either. The current user should probably be instead set into an instance variable, even if this poses problems accessing it from the HTMLGen object H... I'm trying to understand better how scoping works in Sinatra but there is not much documentation about things like how concurrency is handled, scope of different kind of variables/methods and so forth.

antirez avatar Oct 28 '11 16:10 antirez