Register - Login
Views: 90622433
Main - Memberlist - Active users - Calendar - Wiki - IRC Chat - Online users
Ranks - Rules/FAQ - Stats - Latest Posts - Color Chart - Smilies
04-23-18 10:57:19 PM

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

Level: 17


Posts: 28/90
EXP: 22282
For next: 2461

Since: 06-11-16

From: a grave

Since last post: 123 days
Last activity: 123 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: 41


Posts: 321/485
EXP: 463639
For next: 16506

Since: 02-24-13

From: your dreams

Since last post: 49 days
Last activity: 5 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

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


Posts: 683/776
EXP: 1026686
For next: 57154

Since: 02-19-12

From: Everywhere at once.

Since last post: 64 days
Last activity: 64 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: 243


Posts: 21808/23358
EXP: 224299923
For next: 2615697

Since: 07-03-07

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

Since last post: 1 day
Last activity: 1 day

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


Posts: 322/485
EXP: 463639
For next: 16506

Since: 02-24-13

From: your dreams

Since last post: 49 days
Last activity: 5 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

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


Posts: 685/776
EXP: 1026686
For next: 57154

Since: 02-19-12

From: Everywhere at once.

Since last post: 64 days
Last activity: 64 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: 1788
For next: 399

Since: 07-12-16

Pronouns: she or they

Since last post: 84 days
Last activity: 38 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: 22282
For next: 2461

Since: 06-11-16

From: a grave

Since last post: 123 days
Last activity: 123 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: 41


Posts: 327/485
EXP: 463639
For next: 16506

Since: 02-24-13

From: your dreams

Since last post: 49 days
Last activity: 5 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

Level: 243


Posts: 21890/23358
EXP: 224299923
For next: 2615697

Since: 07-03-07

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

Since last post: 1 day
Last activity: 1 day

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.146503 seconds
Script execution time: 0.013960 seconds
Total render time: 0.160463 seconds
Memory used: 786432