Simpleads creates a menu item at admin/content/simpleads/none/stat which links to a blank page only. Accessing the statistics for a single add by clicking on the 'Statistics' link for a particular ad on admin/content/simpleads works as in that case 'none' is replaced with the node id of the ad to look up. This balnk link in the menu is confusing, and makes it apear as though the statistics feature is broken, when actually it works without issue (when accessed at the correct url)

CommentFileSizeAuthor
#1 1970780-simpleads-menu-item.patch2.82 KBmrfelton

Comments

mrfelton’s picture

mrfelton commented
Status: Active » Needs review
StatusFileSize
new 1970780-simpleads-menu-item.patch2.82 KB

Attached patch fixes up the menu item declarations.

proteo’s picture

proteo commented

Hi. Just wanted to report that the patch itself works fine for me (it gets ride of the extra menu entries). However, now I'm seeing this error message whenever I visit admin content screens:

Notice: Undefined offset: 3 en _menu_translate() (línea 777 de /var/www/site/includes/menu.inc).

The error sticks after flushing caches, and even after disabling and deinstalling the admin menu module.

DrCord’s picture

DrCord commented
Issue summary: View changes

I fixed this by adding

if($nid == 'none'){
        return drupal_goto('admin/content/simpleads');
    }

on line 16 of simpleads.stat.inc, before anything else happens.

minnur’s picture

minnur commented
Status: Needs review » Closed (won't fix)