?
avatar brianteeman
brianteeman
11 Nov 2015

In the layout files
/layouts/joomla

Some have
defined('JPATH_BASE') or die;
Some have
defined('_JEXEC') or die;

and readmore has neither

avatar brianteeman brianteeman - open - 11 Nov 2015
avatar brianteeman
brianteeman - comment - 12 Nov 2015

Happy to do a PR to fix this if someone can just say which is the correct option


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8384.

avatar zero-24
zero-24 - comment - 12 Nov 2015

@brianteeman the correct is defined('JPATH_BASE') or die; for layouts. This is my last stand on this ;)

avatar brianteeman
brianteeman - comment - 12 Nov 2015

OK - PR on its way

On 12 November 2015 at 11:42, zero-24 notifications@github.com wrote:

@brianteeman https://github.com/brianteeman the correct is defined('JPATH_BASE')
or die; for layouts. This is my last stand on this ;)


Reply to this email directly or view it on GitHub
#8384 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman
brianteeman - comment - 12 Nov 2015

See #8398 for a PR

avatar brianteeman brianteeman - change - 12 Nov 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-11-12 12:12:41
Closed_By brianteeman
avatar brianteeman brianteeman - close - 12 Nov 2015

Add a Comment

Login with GitHub to post a comment