_reptag_callback() contains the following code:

  switch ($op) {
    case 'tagadmin':
      // ...
    case 'refresh':
      // ...
  }

tagadmin should be tagadd.

CommentFileSizeAuthor
#2 reptag_admin_090456.jpg12.14 KBavpaderno
#2 reptag_admin_090457.jpg14.51 KBavpaderno

Comments

profix898’s picture

profix898 commented

Why do you think that tagadmin is wrong? Can you explain? tagadmin handles add and edit operations, there is no tagadd anywhere else, right?

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
StatusFileSize
new reptag_admin_090457.jpg14.51 KB
new reptag_admin_090456.jpg12.14 KB

I explain myself better.
If you disable the JavaScrupt support for the administration pages, when you try to add a new site or user tag you will see an empty page.

In detail, follow these steps:

  • Disable the JavaScript support from admin/settings/reptag.
  • Add a new site tag by clicking on SiteWide Tags; the path shown in the browser will be admin/settings/reptag/site.
  • Click on the plus image; the path show will be admin/settings/reptag/nojs/tagadd/site?destination=admin%2Fsettings%2Freptag%2Fsite; the page which you will see is a page containing just an empty fieldset.

The parameter passed to the page callback is clearly tagadd, not tagadmin; as the page callback doesn't check if the parameter $op gets such value, then it will not output anything in that case.

See the attachments for a screenshot of the page I see before to change the code as I described.

profix898’s picture

profix898 commented
Title: Wrong code in _reptag_callback() » nojs tags administration points to non-existing path
Status: Active » Fixed

Fixed in CVS. Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.