Added check for -JEXEC

HermanPeeren · HermanPeeren · commit 2eff92fbedd6 · 2025-09-20T10:01:12.000+02:00
(and escape for phpcs)
diff --git a/src/administrator/components/com_ccm/src/Controller/CmsController.php b/src/administrator/components/com_ccm/src/Controller/CmsController.php
@@ -8,6 +8,11 @@
  */
 
 namespace Joomla\Component\CCM\Administrator\Controller;
+
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
+
 use Joomla\CMS\MVC\Controller\FormController;
 class CmsController extends FormController
 {
diff --git a/src/administrator/components/com_ccm/src/Controller/CmssController.php b/src/administrator/components/com_ccm/src/Controller/CmssController.php
@@ -9,7 +9,9 @@
 
 namespace Joomla\Component\CCM\Administrator\Controller;
 
-defined('_JEXEC') or die;
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
 
 use Joomla\CMS\MVC\Controller\ListController;
 use Joomla\CMS\Router\Route as JRoute;
diff --git a/src/administrator/components/com_ccm/src/Controller/DisplayController.php b/src/administrator/components/com_ccm/src/Controller/DisplayController.php
@@ -7,8 +7,13 @@
  * @license     GNU General Public License version 2 or later; see LICENSE.txt
  */
 
-// first file the Joomla! MVC checks after the component bootup
+
 namespace Joomla\Component\CCM\Administrator\Controller;
+
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
+
 use Joomla\CMS\MVC\Controller\BaseController;
 
 class DisplayController extends BaseController
diff --git a/src/administrator/components/com_ccm/src/Controller/MigrationController.php b/src/administrator/components/com_ccm/src/Controller/MigrationController.php
@@ -9,6 +9,10 @@
 
 namespace Joomla\Component\CCM\Administrator\Controller;
 
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
+
 use Joomla\CMS\MVC\Controller\BaseController;
 
 class MigrationController extends BaseController
diff --git a/src/administrator/components/com_ccm/src/Extension/CCMComponent.php b/src/administrator/components/com_ccm/src/Extension/CCMComponent.php
@@ -9,7 +9,9 @@
 
 namespace Joomla\Component\CCM\Administrator\Extension;
 
-defined('_JEXEC') or die;
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
 
 use Joomla\CMS\Extension\BootableExtensionInterface;
 use Joomla\CMS\Extension\MVCComponent;
diff --git a/src/administrator/components/com_ccm/src/Fields/CcmField.php b/src/administrator/components/com_ccm/src/Fields/CcmField.php
@@ -8,14 +8,17 @@
  */
 
 namespace Joomla\Component\CCM\Administrator\Fields;
+
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
+
 use Joomla\CMS\Factory;
 use Joomla\CMS\Form\Field\ListField;
 
-// content in wordpress & text in joomla 
+// content in WordPress & text in Joomla
 // what if data is null (e.g. image), then I can't get its type 
 
-// prefixed the name of our field with our company name. 
-// This helps prevent clashes with other field types defined by other developers.
 class CcmField extends ListField {
     // define a custom form field
     // the name of the type for our new field
diff --git a/src/administrator/components/com_ccm/src/Fields/CmsField.php b/src/administrator/components/com_ccm/src/Fields/CmsField.php
@@ -8,10 +8,13 @@
  */
 
 namespace Joomla\Component\CCM\Administrator\Fields;
+
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
+
 use Joomla\CMS\Form\Field\ListField;
 
-// prefixed the name of our field with our company name. 
-// This helps prevent clashes with other field types defined by other developers.
 class CmsField extends ListField { // this should be named CmsNameField
     // define a custom form field
     // the name of the type for our new field
diff --git a/src/administrator/components/com_ccm/src/Fields/CmsObjTypeField.php b/src/administrator/components/com_ccm/src/Fields/CmsObjTypeField.php
@@ -8,10 +8,13 @@
  */
 
 namespace Joomla\Component\CCM\Administrator\Fields;
+
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
+
 use Joomla\CMS\Form\Field\ListField;
 
-// prefixed the name of our field with our company name. 
-// This helps prevent clashes with other field types defined by other developers.
 class CmsObjTypeField extends ListField {
     // define a custom form field
     // the name of the type for our new field
diff --git a/src/administrator/components/com_ccm/src/Helper/MigrationHelper.php b/src/administrator/components/com_ccm/src/Helper/MigrationHelper.php
@@ -9,6 +9,10 @@
 
 namespace Joomla\Component\CCM\Administrator\Helper;
 
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
+
 use Joomla\CMS\Http\HttpFactory;
 
 /**
diff --git a/src/administrator/components/com_ccm/src/Model/CmsModel.php b/src/administrator/components/com_ccm/src/Model/CmsModel.php
@@ -13,7 +13,9 @@
 use Joomla\CMS\Http\HttpFactory;
 use Joomla\CMS\MVC\Model\AdminModel;
 
+// phpcs:disable PSR1.Files.SideEffects
 \defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
 
 //cms type = wordpress, joomla, drupal, etc.
 //their APIs
diff --git a/src/administrator/components/com_ccm/src/Model/CmssModel.php b/src/administrator/components/com_ccm/src/Model/CmssModel.php
@@ -14,7 +14,9 @@
 use Joomla\CMS\MVC\Factory\MVCFactoryInterface;
 use Joomla\CMS\MVC\Model\ListModel;
 
+// phpcs:disable PSR1.Files.SideEffects
 \defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
 
 class CmssModel extends ListModel {
 
diff --git a/src/administrator/components/com_ccm/src/Model/MigrationModel.php b/src/administrator/components/com_ccm/src/Model/MigrationModel.php
@@ -9,6 +9,10 @@
 
 namespace Joomla\Component\CCM\Administrator\Model;
 
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
+
 use Joomla\CMS\Factory;
 use Joomla\CMS\Filter\OutputFilter;
 use Joomla\CMS\Http\Http;
diff --git a/src/administrator/components/com_ccm/src/Service/Router.php b/src/administrator/components/com_ccm/src/Service/Router.php
@@ -9,7 +9,9 @@
 
 namespace Joomla\Component\CCM\Administrator\Service;
 
-defined('_JEXEC') or die;
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
 
 use Joomla\CMS\Router\ApiRouter;
 use Joomla\CMS\Router\Route;
diff --git a/src/administrator/components/com_ccm/src/Table/CmsTable.php b/src/administrator/components/com_ccm/src/Table/CmsTable.php
@@ -9,7 +9,9 @@
 
 namespace Joomla\Component\CCM\Administrator\Table;
 
-defined('_JEXEC') or die;
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
 
 use Joomla\CMS\Table\Table;
 use Joomla\Database\DatabaseInterface;
diff --git a/src/administrator/components/com_ccm/src/View/Cms/HtmlView.php b/src/administrator/components/com_ccm/src/View/Cms/HtmlView.php
@@ -9,7 +9,9 @@
 
 namespace Joomla\Component\CCM\Administrator\View\Cms;
 
-defined('_JEXEC') or die;
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
 
 use Joomla\CMS\Factory;
 use Joomla\CMS\Helper\ContentHelper;
diff --git a/src/administrator/components/com_ccm/src/View/Cmss/HtmlView.php b/src/administrator/components/com_ccm/src/View/Cmss/HtmlView.php
@@ -9,7 +9,9 @@
 
 namespace Joomla\Component\CCM\Administrator\View\Cmss;
 
-defined('_JEXEC') or die;
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
 
 use Joomla\CMS\Language\Text;
 use Joomla\CMS\MVC\View\HtmlView as BaseHtmlView;
diff --git a/src/administrator/components/com_ccm/src/View/Migration/HtmlView.php b/src/administrator/components/com_ccm/src/View/Migration/HtmlView.php
@@ -9,7 +9,9 @@
 
 namespace Joomla\Component\CCM\Administrator\View\Migration;
 
-defined('_JEXEC') or die;
+// phpcs:disable PSR1.Files.SideEffects
+\defined('_JEXEC') or die;
+// phpcs:enable PSR1.Files.SideEffects
 
 use Joomla\CMS\Language\Text;
 use Joomla\CMS\MVC\View\HtmlView as BaseHtmlView;
@@ -35,7 +37,7 @@ public function display($tpl = null): void
 
         // TODO
         // after each step say echo "Mapping is done" --> then echo "Migration is done"
-        // this can be added in js in frontend in mdeia folder
+        // this can be added in js in frontend in media folder
         // from webassets 
         parent::display($tpl);
     }
diff --git a/src/administrator/components/com_ccm/tmpl/cms/edit.php b/src/administrator/components/com_ccm/tmpl/cms/edit.php
@@ -7,6 +7,8 @@
  * @license     GNU General Public License version 2 or later; see LICENSE.txt
  */
 
+defined('_JEXEC') or die;
+
 use Joomla\CMS\HTML\HTMLHelper;
 use Joomla\CMS\Router\Route;
 
@@ -17,19 +19,19 @@
 ?>   
 
 <form action="<?php echo Route::_('index.php?option=com_ccm&view=cms&layout=edit&id=' . (int) $this->item->id); ?>" method="post" name="adminForm" id="cms-form" class="form-validate">
-    <div>
-        <div class="row">
-            <div class="col-md-9">
-                <div class="row">
-                    <div class="col-md-6">
-                        <?php foreach ($this->form->getFieldset() as $field) :?>
+    <div>
+        <div class="row">
+            <div class="col-md-9">
+                <div class="row">
+                    <div class="col-md-6">
+                        <?php foreach ($this->form->getFieldset() as $field) :?>
                             <?php echo $field->renderField(); ?>
                         <?php endforeach;?>
-                    </div>
-                </div>
-            </div>
-        </div>
-    </div>
-    <input type="hidden" name="task" value="">
-    <?php echo HTMLHelper::_('form.token'); ?>
+                    </div>
+                </div>
+            </div>
+        </div>
+    </div>
+    <input type="hidden" name="task" value="">
+    <?php echo HTMLHelper::_('form.token'); ?>
 </form>
diff --git a/src/administrator/components/com_ccm/tmpl/cmss/default.php b/src/administrator/components/com_ccm/tmpl/cmss/default.php
@@ -7,21 +7,20 @@
  * @license     GNU General Public License version 2 or later; see LICENSE.txt
  */
 
+defined('_JEXEC') or die;
+
 use Joomla\CMS\HTML\HTMLHelper;
 use Joomla\CMS\Language\Text;
 use Joomla\CMS\Layout\LayoutHelper;
 use Joomla\CMS\Router\Route;
 
-defined('_JEXEC') or die;
-
 $listDirn  = $this->escape($this->state->get('list.direction'));
 $listOrder = $this->escape($this->state->get('list.ordering'));
 
 /** @var \Joomla\CMS\WebAsset\WebAssetManager $wa */
 $wa = $this->getDocument()->getWebAssetManager();
 $wa->useScript('table.columns')
     ->useScript('multiselect');
-
 ?>
 
 <form action="<?php echo Route::_('index.php?option=com_ccm&view=cmss'); ?>" method="post" name="adminForm" id="adminForm">
diff --git a/src/administrator/components/com_ccm/tmpl/migration/default.php b/src/administrator/components/com_ccm/tmpl/migration/default.php
@@ -8,6 +8,7 @@
  */
 
 defined('_JEXEC') or die;
+
 use Joomla\CMS\HTML\HTMLHelper;
 use Joomla\CMS\Language\Text;
 use Joomla\CMS\Router\Route;
