Ticket #266 (closed task: fixed)

Opened 7 months ago

Last modified 3 months ago

replacing "no privilege left" msg by something more useful

Reported by: quinox Owned by: daelstorm
Priority: minor Milestone:
Component: nicotine Version: SVN (checked out today)
Keywords: privileges Cc:

Description

in the nicotine room we get a lot of questions about privileges. The official slsk client does not report this kind of thing upon startup at all, so I think it's wise to be a bit more verbose.

My suggestion:

Replace line 801 in file pynicotine/pynicotine.py

current msg:

if msg.seconds == 0:

self.logMessage(_("You have no privileges left"))

proposed msg:

if msg.seconds == 0:

self.logMessage(_("You have no privileges left. They are not needed for Nicotine to function properly. You can acquire privileges by donating at %s") % ("hŧŧp://www.slsknet.org/userlogin.php?username="+self.config.sections["server"]["login"].replace(' ','+')))

The replace() is needed to make the link clickable in case the user has spaces in its name

Attachments

Change History

Changed 7 months ago by quinox

On a side note, the link doesn't work if the nick has " in them - perhaps they need to be form encoded. Also note that usernames with <> will screw up the slsknet.org page since that page is XSS exploitable

Changed 3 months ago by quinox

  • status changed from new to closed
  • resolution set to fixed

Committed

Add/Change #266 (replacing "no privilege left" msg by something more useful)

Author



Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.