looking at it now:
* Changed folder name xxxjrAutoNotify to xxAutoNotify
* Renamed all instances of jrAutoNotify everywhere in the files to xxAutoNotify to match folder name
* index.php contains a view called view_xxAutoNotify() there is no view url specified, needs changing.
Anything in the index.php file is a view used to take control of a url, so since the module wants to take control of the 'auto-notify' url any url found at site.com/
auto-notify will look for function in index.php and if its not there, it will look for a file index.tpl in the /modules/xxAutoNotify/templates/index.tpl
The URL that the module wants to take control of is found in the _meta() function in include.php, here:
function xxAutoNotify_meta(){
$_tmp = array(
'name' => 'Auto Notify',
'url' => 'auto-notify',
.......
}
To take control of a specific url, like site.com/auto-notify/speedway that would correspond to
index.php:
function view_xxAutoNotify_speedway($_post, $_user, $_conf)
{
return 'I like going to the speedway';
}
You cant have a view function without a view specified so
view_xxAutoNotify() won't work, if you want it to be the default for all non defined views, then use
view_xxAutoNotify_default.
--
* adding a changelog.txt and incrementing the version to 1.0.0
* removed the hyphen in auto-notify url because I worry it could be interpreted incorrectly
(personal preference not a requirement)
--
have just installed the module: The index.php file is not needed at all, there are no views for this module. Deleted it.
--
* config.php is good, it just needs a bit of cleaning up, it would have worked though.
--
Tested, seams to be good, have added a requirement for the jrForum module since it wont do anything without that and added a check before trying to write to the datastore.
Other than that, well done.
Attached
updated by @michael: 11/20/16 11:22:51PM