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

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

Level: 18


Posts: 28/90
EXP: 25029
For next: 4868

Since: 06-11-16

From: a grave

Since last post: 301 days
Last activity: 116 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/500
EXP: 507749
For next: 13613

Since: 02-24-13

From: your dreams

Since last post: 2 days
Last activity: 1 day

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: 1066478
For next: 17362

Since: 02-19-12

From: Everywhere at once.

Since last post: 243 days
Last activity: 243 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: 229307731
For next: 879535

Since: 07-03-07

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

Since last post: 7 hours
Last activity: 3 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/500
EXP: 507749
For next: 13613

Since: 02-24-13

From: your dreams

Since last post: 2 days
Last activity: 1 day

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: 1066478
For next: 17362

Since: 02-19-12

From: Everywhere at once.

Since last post: 243 days
Last activity: 243 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: 2018
For next: 169

Since: 07-12-16

Pronouns: they/them

Since last post: 263 days
Last activity: 36 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: 25029
For next: 4868

Since: 06-11-16

From: a grave

Since last post: 301 days
Last activity: 116 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/500
EXP: 507749
For next: 13613

Since: 02-24-13

From: your dreams

Since last post: 2 days
Last activity: 1 day

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: 229307731
For next: 879535

Since: 07-03-07

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

Since last post: 7 hours
Last activity: 3 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.213825 seconds
Script execution time: 0.013869 seconds
Total render time: 0.227694 seconds
Memory used: 1048576