photo

Eduardo Marques

shared this idea
5 years ago

Statistics

0
Comments
776
Views

Relates to

Share

3
votes

Problems at Hook System

ref. O-R v.3.2.7

Attached you have the hook file I used to perform tests.

Found the following problems using IExplorer v.11:

function detect_mobile_browser($user_agent)

- this function is being called 8(!) times on each O-R action (notice: this was detected while using a PC)

function after_user_logout($userID)

- variable $userID is being UNSET before this function is executed so this var parses a null value

function befure_user_change($userID, $postvars)

- $postvars value is not being parsed

- logged as admin, after saving changes at user account a message is displayed: Warning: Missing argument 2 for publicrecords_hook::befure_user_change(), called in E:\wamp\www\or_latest\include\hooks.inc.php on line 43 and defined in E:\wamp\www\or_latest\hooks\publicrecords_hook.php on line...

- also notice typo at function name

Under Consideration
+1 I like this idea

Leave Comment

photo

Attach files...

The file must be a jpg, gif, png, bmp, ico, pdf, doc, rtf, txt, zip or rar no more than 20M