AMP
December 10, 2018, 12:15:59 *
Welcome, Guest. Please login or register.

Login with username, password and session length
 
   Home   Help Search Login Register  
Pages: [1]   Go Down
  Print  
Author Topic: group names with '&' can't be clicked  (Read 4962 times)
0 Members and 1 Guest are viewing this topic.
Asle
Administrator
Hero Member
*****
Offline Offline

Posts: 1 013


View Profile WWW
« on: August 26, 2009, 21:35:35 »

as reported some time ago, groups containing '&' or weird symbols, can't be clicked (to get the complete list of authors in this group).
This is a major problem, using group names in links, instead of IDs. Fixing this is not easy, as some rewriting of somepart of the core of AMP is needed. Still, I'm on it Smiley

Sylvain
Logged
Asle
Administrator
Hero Member
*****
Offline Offline

Posts: 1 013


View Profile WWW
« Reply #1 on: August 27, 2009, 20:45:32 »

done.
Please, report problems if you find any.
Logged
Crown
Administrator
Hero Member
*****
Offline Offline

Posts: 556


Cryptoburners^AMP


View Profile WWW
« Reply #2 on: August 28, 2009, 10:58:37 »

Works fine.  Smiley

I like the group+acronym+logo page. congrats!
Logged
Asle
Administrator
Hero Member
*****
Offline Offline

Posts: 1 013


View Profile WWW
« Reply #3 on: August 28, 2009, 11:57:30 »

right, well, this is mostly Monty's work, so ...
Logged
deus-ex
Sr. Member
****
Offline Offline

Posts: 478



View Profile
« Reply #4 on: August 28, 2009, 20:50:16 »

This is not an attempt to lessen the efforts that have been invested into this issue, but isn't replacing "&" with "and" in group names just kind of a hack/clutch rather than a real fix as it doesn't preserve the original spelling? Roll Eyes Somehow I don't feel comfortable with this. Then again probably other solutions are to costly to implement?
Logged

Best regards
deus-ex
Asle
Administrator
Hero Member
*****
Offline Offline

Posts: 1 013


View Profile WWW
« Reply #5 on: August 28, 2009, 21:31:28 »

I'm not sure I understand. The "fix" was to trash to PHP code that provided links with group names, instead of ID, which should have been done in the first place. This was certainly no cheap update, as I spent two evenings on this one. Some code from the core of AMP had to be rewritten. So, that makes it the "costly to implement" solution, from my point of view.
Then again, I may have misunderstood what you said.
Still, that was fun, as I spent all the time IRC-ing with Monty Smiley. Good time.
Logged
deus-ex
Sr. Member
****
Offline Offline

Posts: 478



View Profile
« Reply #6 on: August 29, 2009, 09:40:44 »

For example searching for group CNCD gives following result:

  Now: Carillon and Cyberiad (CNCD)
 Then: Carillon & Cyberiad (CNCD) <- original spelling

Got me now? Smiley
Logged

Best regards
deus-ex
Asle
Administrator
Hero Member
*****
Offline Offline

Posts: 1 013


View Profile WWW
« Reply #7 on: August 29, 2009, 12:08:21 »

That was due to one of the admin, changing the name. It's unrelated to the present update. I've changed the name again and you can see there's a & again. It doesn't affect the way it works at all.
Any other wrong name you've noticed ?
Just quick search for '&' and some result appear. No many, right, but that's not my doing Smiley. Searching for ' and ' brings more result. If a name is wrong, just tell me

And thanks for your feedback ! Smiley
« Last Edit: August 29, 2009, 12:13:04 by Asle » Logged
deus-ex
Sr. Member
****
Offline Offline

Posts: 478



View Profile
« Reply #8 on: August 29, 2009, 13:09:29 »

That was due to one of the admin, changing the name.

That admin is being punished/tortured as we speak, I presume? Grin


Any other wrong name you've noticed ?

Checking group names containing a "&" character I found these to be affected, too:

Defpaccers & Paranoid (DPAD)
Dual Crew & Shining (DCS)
Hugo & Mamuttene (H^M)
Panic & Agnostic Front (PAF)
Success & The Ruling Company (SCS*TRC)
Tristar & Red Sector Inc. (TRSI)
Logged

Best regards
deus-ex
Asle
Administrator
Hero Member
*****
Offline Offline

Posts: 1 013


View Profile WWW
« Reply #9 on: August 29, 2009, 13:58:10 »

all renamed. Thanks !
I'll have a few words with this admin, once he is within reach Smiley.
Logged
Pages: [1]   Go Up
  Print  
 
Jump to:  

Powered by SMF | SMF © 2015, Simple Machines