Reminder for developers of modules or other api
Posted: Wed May 24, 2017 12:41 am
In your development code :
The references to directories should never be made literally,
But always by using a variable or a constant, especially when they already exist.
Example of error which I find in some modules : (microtiny, tinymce...)
$output.='<__script__ or link src=$config->smart_root_url().'/tmp/cache/'.....etc
/tmp/cache was defined in the config.php file as a variable, it may therefore vary !
instead of : $config->smart_root_url().'/tmp/cache/'
use simply : $config['public_cache_url'] (Or the appropriate variable)
Moreover, keep in mind the difference between
$config['public_cache_url'] (public access chmod 705)
$config['tmp_cache_location'] or $config['tmp_templates_c_location'] (internal processing no public access chmod 700)
otherwise, variables defined in the config file are useless !
Lastly, If your directory should not be customizable, like the directory "Lib", use a constant.
Thank you all,
The references to directories should never be made literally,
But always by using a variable or a constant, especially when they already exist.
Example of error which I find in some modules : (microtiny, tinymce...)
$output.='<__script__ or link src=$config->smart_root_url().'/tmp/cache/'.....etc
/tmp/cache was defined in the config.php file as a variable, it may therefore vary !
instead of : $config->smart_root_url().'/tmp/cache/'
use simply : $config['public_cache_url'] (Or the appropriate variable)
Moreover, keep in mind the difference between
$config['public_cache_url'] (public access chmod 705)
$config['tmp_cache_location'] or $config['tmp_templates_c_location'] (internal processing no public access chmod 700)
otherwise, variables defined in the config file are useless !
Lastly, If your directory should not be customizable, like the directory "Lib", use a constant.
Thank you all,