I get an error

it doesn't run on PHP 5.4.

- we need to change the module logo to our standard XOOPS "Blue Move" logo. If you need help, let me know

Did you measure how his module impacts overall performance on a busy site (like what you have)?

- add "Purge" tab, to be able to delete selected entries manually (with option to delete, or export and then delete). Right now you can only set it in Preferences

- add "Export" tab, to export as CSV or Excel files for selected entries (like selected users or selected dates)

- and for release 2.0 or 3.0, it would be interesting to link Userlog to entries in Protector, to for example, select a suspicious activity in Protector, and then see all entries for the same IP from the Userlog. Or the opposite - select IP or User in Userlog, and see if there were any entries in the Protector.

I want to invite Mage, Trabis, Mamba, timgno, zyspec, red, voltan, mojtaba, peekey, black_beard, alfred and all other developers that i cannot remember there names to come here and share in developments. please lets make it a community project.

this module should work in 5.4 what is the issue?

Fatal error: require_once(): Failed opening required 'PEAR/Config.php' (include_path='.;s:\wamp64\bin\php\php5.4.8\pear') in S:\wamp64\bin\php\php5.4.8\PEAR\Common.php on line 836
Call Stack
# Time Memory Function Location
1 0.0012 306224 {main}( ) ..\index.php:0
2 0.1075 5529800 xoops_cp_header( ) ..\index.php:25
3 0.1097 5745144 XoopsGuiDefault->header( ) ..\cp_functions.php:29
4 0.1097 5747312 XoopsSystemGui->header( ) ..\default.php:51
5 0.1206 6431864 XoopsPreload->triggerEvent( ) ..\gui.php:90
6 0.1206 6432848 call_user_func ( ) ..\preload.php:140
7 0.1207 6432880 UserlogCorePreload::eventSystemClassGuiHeader( ) ..\preload.php:140
8 0.1209 6448072 include( 'S:\wamp64\www\xtest256a6\modules\userlog\include\log.php' ) ..\core.php:42
9 0.1223 6602920 include_once( 'S:\wamp64\bin\php\php5.4.8\PEAR\Common.php' ) ..\log.php:28

I call it Alpha because im not sure about anything in this module like the structures and features. nothing is final in this module. once we finalize all my doubts about this module we can release a final version soon.

we need to change the module logo

- in Help, the name of the module should provide a link back to the module

the ASC ordering in column titles doesn't work - it is always DESC

what is the difference between logs in the DB and logs in the file.

developers help for userlog module.

1- log user activities structure.
I assume we need to gather below information from user. If you have any idea about them please let me know.
i dont sanitize them because i want to log them as ther are. what is your idea. Im a very basic programmer. please help me.

in userlog/class/setting.php

public function getOptions($option = null, $V = "value")
    {
        $V = strtolower($V);
        
        if ($this->userlog->getUser()) {
            $uid = $this->userlog->getUser()->getVar('uid');
            $uname = $this->userlog->getUser()->getVar('uname');
            $last_login = $this->userlog->getUser()->getVar('last_login');
            $admin = $this->userlog->getUser()->isAdmin();
            $groups = "g" . implode("g",$this->userlog->getUser()->getGroups()); // g1g2
        } else {
            $uid = 0;
            $uname = '';
            $last_login = 0;
            $admin = 0;
            $groups = "g" . XOOPS_GROUP_ANONYMOUS; // g3
        }
        $options = array(
        "log_id" =>        array(    "type" => "int",
                                "title" => _AM_USERLOG_LOG_ID,
                                "value" => null // null for now
                                ),
        "log_time" =>    array(    "type" => "int",
                                "title" => _AM_USERLOG_LOG_TIME,
                                "value" => time()
                                ),
        "uid" =>         array(    "type" => "int",
                                "title" => _AM_USERLOG_UID,
                                "value" => $uid
                                ),
        "uname" =>        array(    "type" => "text",
                                "title" => _AM_USERLOG_UNAME,
                                "value" => $uname
                                ),
        "admin" =>         array(    "type" => "bool",
                                "title" => _AM_USERLOG_ADMIN,
                                "value" => $admin
                                ),
        "groups" =>        array(    "type" => "text",
                                "title" => _AM_USERLOG_GROUPS,
                                "value" => $groups
                                ),
        "last_login" =>    array(    "type" => "int",
                                "title" => _AM_USERLOG_LAST_LOGIN,
                                "value" => $last_login
                                ),
        "user_ip" =>    array(    "type" => "text",
                                "title" => _AM_USERLOG_USER_IP,
                                "value" => $_SERVER['REMOTE_ADDR']
                                ),
        "user_agent" => array(    "type" => "text",
                                "title" => _AM_USERLOG_USER_AGENT,
                                "value" => $_SERVER['HTTP_USER_AGENT']
                                ),
        "url" =>         array(    "type" => "text",
                                "title" => _AM_USERLOG_URL,
                                "value" => $_SERVER['REQUEST_URI']
                                ),
        "script" =>     array(    "type" => "text",
                                "title" => _AM_USERLOG_SCRIPT,
                                "value" => end(explode('/',$_SERVER['PHP_SELF']))
                                ),
        "referer" =>     array(    "type" => "text",
                                "title" => _AM_USERLOG_REFERER,
                                "value" => !empty($_SERVER['HTTP_REFERER']) ? $_SERVER['HTTP_REFERER'] : ""
                                ),
        "pagetitle" =>     array(    "type" => "text",
                                "title" => _AM_USERLOG_PAGETITLE,
                                "value" => isset($GLOBALS['xoopsTpl']) ? $GLOBALS['xoopsTpl']->get_template_vars("xoops_pagetitle") : ""
                                ),
        "module" =>     array(    "type" => "text",
                                "title" => _AM_USERLOG_MODULE,
                                "value" => $this->userlog->getLogModule()->getVar("dirname")
                                ),
        "item_name" =>     array(    "type" => "text",
                                "title" => _AM_USERLOG_ITEM_NAME,
                                "value" => null
                                ),
        "item_id" =>     array(    "type" => "text",
                                "title" => _AM_USERLOG_ITEM_ID,
                                "value" => null
                                ),
        // user data input method
        "request_method" => array(    "type" => "text",
                                    "title" => _AM_USERLOG_REQUEST_METHOD,
                                    "value" => $_SERVER['REQUEST_METHOD']
                                    ),
        "get" =>         array(    "type" => "text",
                                "title" => _AM_USERLOG_GET,
                                "value" => $_GET
                                ),
        "post" =>         array(    "type" => "text",
                                "title" => _AM_USERLOG_POST,
                                "value" => $_POST
                                ),
        "request" =>    array(    "type" => "text",
                                "title" => _AM_USERLOG_REQUEST,
                                "value" => $_REQUEST
                                ),
        "files" =>        array(    "type" => "text",
                                "title" => _AM_USERLOG_FILES,
                                "value" => $_FILES
                                ),
        "env" =>         array(    "type" => "text",
                                "title" => _AM_USERLOG_ENV,
                                "value" => $_ENV
                                ),
        "session" =>     array(    "type" => "text",
                                "title" => _AM_USERLOG_SESSION,
                                "value" => $_SESSION
                                ),
        "cookie" =>     array(    "type" => "text",
                                "title" => _AM_USERLOG_COOKIE,
                                "value" => $_COOKIE
                                ),
        "header" =>     array(    "type" => "text",
                                "title" => _AM_USERLOG_HEADER,
                                "value" => headers_list()
                                ),
        "logger" =>     array(    "type" => "text",
                                "title" => _AM_USERLOG_LOGGER,
                                "value" => $GLOBALS['xoopsLogger']->errors
                                ),
        // settings will not be logged
        "store_file" => array(    "type" => "int",
                                "title" => _AM_USERLOG_STORE_FILE,
                                "value" => 1
                                ),
        "store_db" =>     array(    "type" => "int",
                                "title" => _AM_USERLOG_STORE_DB,
                                "value" => 1
                                ),
        "views" =>         array(    "type" => "int",
                                "title" => _AM_USERLOG_VIEWS,
                                "value" => 1 // for item_name and item_id
                                ),
        );
        $ret = $this->userlog->getFromKeys($options, $option);
        if (empty($V)) return $ret;
        if ($V == "key") return array_keys($ret);
        $ret2 = null;
        foreach ($ret as $option=>$val) {
            $ret2[$option] = $val[$V];
        }
        return $ret2;
    }

2- setting structure.
i think we just need to log users based on "uid" or "gid" or "ip" and it will cover all possibilities. please let me know your idea.

the table structure is:
in userlog/sql/mysql.sql

CREATE TABLE mod_userlog_set (
  set_id mediumint(8) unsigned NOT NULL auto_increment,
  name varchar(100) NOT NULL default '',
  logby varchar(10) NOT NULL default '',
  unique_id int(11) unsigned NOT NULL default 0,
  options TEXT NOT NULL,
  scope TEXT NOT NULL,
  PRIMARY KEY  (set_id),
  UNIQUE logby_id (logby, unique_id)
) ENGINE=MyISAM;

The above is clear. more information in module help.

3- performance.
it checks every hit to see if the visitor have a setting or not. for the best performance i wrote a get function like this.
in userlog/class/setting.php

public function get()
    {
        // if uid setting exist in File
        $unique_uid = ($this->userlog->getUser()) ? $this->userlog->getUser()->getVar('uid') : 0;
        if ($options = $this->getFile('uid', $unique_uid)) {
            return $options;
        }

        // if gid setting exist in File
        $unique_gid = ($this->userlog->getUser()) ? $this->userlog->getUser()->getGroups() :  array(XOOPS_GROUP_ANONYMOUS);
        foreach ($unique_gid as $gid) {
            if ($options = $this->getFile('gid', $gid)) {
                return $options;
            }
        }
        // if ip setting exist in File
        $unique_ip = XoopsUserUtility::getIP(); // ip as int
        if ($options = $this->getFile('ip', $unique_ip)) {
            return $options;
        }
        // if all exist in File
        if ($options = $this->getFile('all', 0)) {
            return $options;
        }
        ///////////////////////////////////////////////////////////
        // database get All is better for performance???
        $logsetsObj = $this->userlog->getHandler('setting')->getAll();
        if (empty($logsetsObj)) return false; // if not set in db return false
        $uid_unique_uid = "uid" . $unique_uid;
        foreach($unique_gid as $key=>$gid) {
            $gid_unique_gid[$key] = "gid" . $gid;
        }
        $ip_unique_ip = "ip" . $unique_ip;
        foreach($logsetsObj as $setObj) {
            $sLobgy = $setObj->logby();
            $sUnique_id    = $setObj->unique_id();
            $sLogbyId = $sLobgy . $sUnique_id;
            // if uid setting exist in db return it            
            if($sLogbyId == $uid_unique_uid ||
            // if gid setting exist in db return it
               in_array($sLogbyId, $gid_unique_gid) ||
            // if ip setting exist in db return it
               $sLogbyId == $ip_unique_ip) {
                    $sets = array($setObj->options(), $setObj->scope());
                    $this->setFile($sLobgy, $sUnique_id, $sets); // build cache
                    return $sets;
            }
            // if all exist in db
            if($sUnique_id == 0) {
                $sets = array($setObj->options(), $setObj->scope());
                $this->setFile('all', 0, $sets); // build cache
                return $sets;
            }
        }
        return false;
    }

as you can see it create cache files for each settings. IMO it will help to reduce queries for users have a setting.

for getting settings from db i firstly used criteria but then i decide to get all settings at once and then find if the user have setting or not. IMO it has the better performance. please let me know your opinion. what is the best performance? this function is important for your review.

4- views
now every module have a counter for its item views. eg: news, publisher, page in xoops 2.6
but this counter is useless. we dont need forever views. we need views in a specific time period.
in every news agency the first and top block is "today hot news" and it means news that had been most viewed today.

for a long time that was an unsolved issue for me and i think in the whole xoops we have this problem in all modules.
after starting userlog for a different purpose I suddenly reach to an innovative idea for gathering all views in all modules.
i used module "notification" because every module item that should have views have a notification too.
there we have all needed information about items. so i wrote this nice and tricky function:

public function setItem()
    {
        $not_config =& $this->userlog->getLogModule()->getInfo('notification');
        if (!empty($not_config)) {
            foreach ($not_config['category'] as $category) {
                // if $item_id != 0 ---> return true
                if (!empty($category['item_name']) && $item_id = UserlogRequest::getInt($category['item_name'], 0)){
                    $this->setVar('item_name', $category['item_name']);
                    $this->setVar('item_id', $item_id);
                    return true;
                }
            }
        }
        return false;        
    }

so for example in news we only have these typical URLs:

news/article.php?storyid=ITEM_ID
news/index.php|article.php?storytopic=ITEM_ID

as you can see i just need to store item_name and item_id because i have the module dirname and script name!!!

IMO we should stick to this idea to have a overall item views in the XOOPS and drop all counters in all modules.
i need your idea about this.

Quote:

the ASC ordering in column titles doesn't work - it is always DESC

I cannot reproduce it. where? in admin/logs.php everything is ok.