Register - Login
Views: 88301864
Main - Memberlist - Active users - Calendar - Wiki - IRC Chat - Online users
Ranks - Rules/FAQ - JCS - Stats - Latest Posts - Color Chart - Smilies
01-19-18 08:09:02 AM

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

Level: 17


Posts: 28/90
EXP: 20677
For next: 4066

Since: 06-11-16
From: a grave

Since last post: 28 days
Last activity: 28 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: 40


Posts: 321/473
EXP: 435187
For next: 6122

Since: 02-24-13
From: your dreams

Since last post: 2 days
Last activity: 19 hours

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

Lantern Ghost
Don't mind me. Just passing through.
Level: 51


Posts: 683/763
EXP: 979786
For next: 34152

Since: 02-19-12
From: Everywhere at once.

Since last post: 5 days
Last activity: 13 hours

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






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: 40


Posts: 322/473
EXP: 435187
For next: 6122

Since: 02-24-13
From: your dreams

Since last post: 2 days
Last activity: 19 hours

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

Lantern Ghost
Don't mind me. Just passing through.
Level: 51


Posts: 685/763
EXP: 979786
For next: 34152

Since: 02-19-12
From: Everywhere at once.

Since last post: 5 days
Last activity: 13 hours

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

Level: 7


Posts: 6/14
EXP: 1235
For next: 213

Since: 07-12-16


Since last post: 1.0 years
Last activity: 7 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: 20677
For next: 4066

Since: 06-11-16
From: a grave

Since last post: 28 days
Last activity: 28 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: 40


Posts: 327/473
EXP: 435187
For next: 6122

Since: 02-24-13
From: your dreams

Since last post: 2 days
Last activity: 19 hours

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






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 a6a22e7 [2018-01-14]
©2000-2018 Acmlm, Xkeeper, Inuyasha, et al.

29 database queries, 4 query cache hits.
Query execution time: 0.151625 seconds
Script execution time: 0.012528 seconds
Total render time: 0.164153 seconds