Register - Login
Views: 96237933
Main - Memberlist - Active users - Calendar - Wiki - IRC Chat - Online users
Ranks - Rules/FAQ - Stats - Latest Posts - Color Chart - Smilies
12-16-18 07:14:36 PM

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

Level: 18


Posts: 28/90
EXP: 25869
For next: 4028

Since: 06-11-16

From: a grave

Since last post: 360 days
Last activity: 174 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: 43


Posts: 321/509
EXP: 528882
For next: 36164

Since: 02-24-13

From: your dreams

Since last post: 4 days
Last activity: 4 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: 1079249
For next: 4591

Since: 02-19-12

From: Everywhere at once.

Since last post: 301 days
Last activity: 301 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: 251


Posts: 21808/24727
EXP: 251528276
For next: 2511600

Since: 07-03-07

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

Since last post: 15 hours
Last activity: 2 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: 43


Posts: 322/509
EXP: 528882
For next: 36164

Since: 02-24-13

From: your dreams

Since last post: 4 days
Last activity: 4 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: 1079249
For next: 4591

Since: 02-19-12

From: Everywhere at once.

Since last post: 301 days
Last activity: 301 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: 2088
For next: 99

Since: 07-12-16

Pronouns: they/them

Since last post: 321 days
Last activity: 56 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: 18


Posts: 37/90
EXP: 25869
For next: 4028

Since: 06-11-16

From: a grave

Since last post: 360 days
Last activity: 174 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: 43


Posts: 327/509
EXP: 528882
For next: 36164

Since: 02-24-13

From: your dreams

Since last post: 4 days
Last activity: 4 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: 251


Posts: 21890/24727
EXP: 251528276
For next: 2511600

Since: 07-03-07

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

Since last post: 15 hours
Last activity: 2 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 6fc366a [2018-12-11]
©2000-2018 Acmlm, Xkeeper, Inuyasha, et al.

29 database queries, 4 query cache hits.
Query execution time: 0.191555 seconds
Script execution time: 0.014026 seconds
Total render time: 0.205581 seconds