Register - Login
Views: 94247404
Main - Memberlist - Active users - Calendar - Wiki - IRC Chat - Online users
Ranks - Rules/FAQ - Stats - Latest Posts - Color Chart - Smilies
08-17-18 01:27:27 PM

Jul - Meta - announcements bug New poll - New thread - Thread closed
Next newer thread | Next older thread
skyu

Level: 17


Posts: 28/90
EXP: 24099
For next: 644

Since: 06-11-16

From: a grave

Since last post: 239 days
Last activity: 53 days

Posted on 08-11-16 01:40:57 PM Link
alright, so in all forums, if I click on Xkeeper's userlink from the announcement bar, it takes me to user profile 34, not 1. said profile doesn't exist anymore, so I don't know who's it is.

anyone else receiving this problem?
StapleButter
Member
Level: 42


Posts: 321/489
EXP: 483571
For next: 37791

Since: 02-24-13

From: your dreams

Since last post: 35 days
Last activity: 2 days

Posted on 08-11-16 01:44:08 PM Link
looks like it's picking the announcement ID instead of the user ID. good old 'id' confusion.
Cuber456

Fuzzy
Don't mind me. Just passing through.
Level: 52


Posts: 683/776
EXP: 1052665
For next: 31175

Since: 02-19-12

From: Everywhere at once.

Since last post: 180 days
Last activity: 180 days

Posted on 08-11-16 05:54:44 PM Link
Honestly, I wouldn't worry about it.

It makes sense. Not sure how JUL does it but I'd guess that every time a user signs up, their ID is +1 greater from the previous user. Hence, all IDs are unique. For whatever reason, the admins got rid of or blocked ID 1 and so forth but kept using the same algorithm to assign IDs.
Xkeeper

Level: 244


Posts: 21808/23358
EXP: 227560366
For next: 2626900

Since: 07-03-07

Pronouns: they/them, she/her, etc.

Since last post: 2 days
Last activity: 21 hours

Posted on 08-11-16 06:03:25 PM Link
You're misunderstanding the problem; the announcement link is using the wrong ID number (the announcement ID) when it should be using the member ID.

It's probably a dumb typo somewhere. It's been a known bug forever but patching bugs is suffering.

Maybe someone should report it on the github repo
StapleButter
Member
Level: 42


Posts: 322/489
EXP: 483571
For next: 37791

Since: 02-24-13

From: your dreams

Since last post: 35 days
Last activity: 2 days

Posted on 08-11-16 06:07:40 PM Link
you're right there, but they didn't get rid of ID 1. Xkeeper is still ID 1.

the following query is used to retrieve the last announcement and its associated metadata:

SELECT announcements.id,user,date,announcements.title,name,sex,powerlevel FROM announcements,users WHERE forum=0 AND user=users.id ORDER BY date DESC LIMIT 1

so you get an array with the following fields:

id -> announcement ID
user -> user ID
date -> date it was posted at
title -> announcement title
name -> user name
sex -> user sex
powerlevel -> user powerlevel

the data is passed to getuserlink() to get the userlink that is displayed. That function expects the 'id' field to be the user ID.
Cuber456

Fuzzy
Don't mind me. Just passing through.
Level: 52


Posts: 685/776
EXP: 1052665
For next: 31175

Since: 02-19-12

From: Everywhere at once.

Since last post: 180 days
Last activity: 180 days

Posted on 08-11-16 06:23:10 PM Link
Whoops. I'm like half asleep right now.
Octangula

Level: 8


Posts: 6/17
EXP: 1940
For next: 247

Since: 07-12-16

Pronouns: they/them

Since last post: 200 days
Last activity: 50 days

Posted on 08-12-16 09:08:00 PM Link
I hate to admit it, but even with my experience with computers, SQL still feels effing terrifyingly like a language that makes less sense than English (which, like Java, is a language of exceptions)...

(And talking of language, is there a policy on swearing here? I usually wouldn't censor myself there, but feels better to ask before putting my foot in it...)
skyu

Level: 17


Posts: 37/90
EXP: 24099
For next: 644

Since: 06-11-16

From: a grave

Since last post: 239 days
Last activity: 53 days

Posted on 08-12-16 10:01:51 PM Link
there's no policy on swearing, no

just.. look at old posts from 2007
StapleButter
Member
Level: 42


Posts: 327/489
EXP: 483571
For next: 37791

Since: 02-24-13

From: your dreams

Since last post: 35 days
Last activity: 2 days

Posted on 08-13-16 03:57:51 AM Link
anyway, either swear openly or don't swear at all-- self-censoring looks dumb imo


on a more on-topic note, I sent a pull request for this matter... wait until it's merged in, I guess
Xkeeper

Level: 244


Posts: 21890/23358
EXP: 227560366
For next: 2626900

Since: 07-03-07

Pronouns: they/them, she/her, etc.

Since last post: 2 days
Last activity: 21 hours

Posted on 12-22-16 11:34:18 PM Link
It is merged in. It only took a few months.
Next newer thread | Next older thread
Jul - Meta - announcements bug New poll - New thread - Thread closed




Rusted Logic

Acmlmboard - commit 5d36857 [2018-03-03]
©2000-2018 Acmlm, Xkeeper, Inuyasha, et al.

29 database queries, 4 query cache hits.
Query execution time: 0.173521 seconds
Script execution time: 0.013623 seconds
Total render time: 0.187144 seconds
Memory used: 786432