Advanced search  

News:

cpg1.5.48 Security release - upgrade mandatory!
The Coppermine development team is releasing a security update for Coppermine in order to counter a recently discovered vulnerability. It is important that all users who run version cpg1.5.46 or older update to this latest version as soon as possible.
[more]

Pages: [1]   Go Down

Author Topic: admin comment switching linked to TOP MENU Comments Link  (Read 2675 times)

0 Members and 1 Guest are viewing this topic.

pinpoint222

  • Contributor
  • Coppermine regular visitor
  • ***
  • Offline Offline
  • Gender: Male
  • Posts: 77
    • The Rawalpindi Website
admin comment switching linked to TOP MENU Comments Link
« on: December 21, 2005, 11:14:52 am »

HI

i just noticed that one can switch On or Off the "comments allowed " as Admin. But when you Turn off comment then the Top menu still has "last comments" Link On. one can remove it from template ofcos but can this be linked to Comments On Off feature in upcoming CPG ? So one doesnt have to edit template everytime comments are diabled for temporary time like couple of weeks?


 
« Last Edit: December 22, 2005, 06:08:41 am by GauGau »
Logged

Tranz

  • Dev Team member
  • Coppermine addict
  • ****
  • Country: 00
  • Offline Offline
  • Gender: Female
  • Posts: 6149
Re: admin comment switching linked to TOP MENU Comments Link
« Reply #1 on: December 21, 2005, 02:51:58 pm »

Interesting idea but I don't think it's feasible. Comments permissions are not global as they vary for usergroups globally and by albums. Just because people can't post comments doesn't mean they shouldn't be able to read other comments.
Logged

Joachim Müller

  • Dev Team member
  • Coppermine addict
  • ****
  • Offline Offline
  • Gender: Male
  • Posts: 47843
  • aka "GauGau"
    • gaugau.de
Re: admin comment switching linked to TOP MENU Comments Link
« Reply #2 on: December 22, 2005, 06:08:25 am »

you'd have to run an extra query to find out if comments exist within the page shown, which would be resources-consuming. Imo it's easier to disable the menu entry as you suggested in the theme.
Logged
Pages: [1]   Go Up
 

Page created in 0.016 seconds with 20 queries.