newicon / neon

neon/cms/migrations/m140506_102106_cms_init.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 			'layout' => 'string COMMENT "the name of the layout template to load for this page"',
 			'template' => 'string COMMENT "the name of the template file"'
 		]);
-		$this->createTable(CmsUrl::tableName(),[
+		$this->createTable(CmsUrl::tableName(), [
 			'id' => 'pk',
 			'url' => 'string',
 			'page_id' => 'int',

neon/cms/migrations/deprecatedMigrations/m170106_131700_cms_page.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
 		$this->run('Generating Page UUIDs and nice_ids', function() {
 			// the soft delete column has not been added yet - so lets turn off this behaviour!
 			CmsPage::$softDelete = false;
-			foreach(CmsPage::find()->each(100) as $page) {
+			foreach (CmsPage::find()->each(100) as $page) {
 				$page->id = CmsPage::uuid64();
 				$page->nice_id = "page_{$page->old_id}";
 				$page->save();
@@ -92,7 +92,7 @@ 
 			"UPDATE $cms_url INNER JOIN $cms_page ON $cms_page.nice_id = $cms_url.page_id SET `page_id` = $cms_page.old_id"
 		);
 		$this->execute("UPDATE $cms_page SET `id` = `old_id`");
-		$this->alterColumn($cms_page, 'id', $this->integer(11) . ' NOT NULL AUTO_INCREMENT');
+		$this->alterColumn($cms_page, 'id', $this->integer(11).' NOT NULL AUTO_INCREMENT');
 		$this->dropColumn($cms_page, 'old_id');
 		$this->dropColumn($cms_page, 'nice_id');
 		$this->alterColumn($cms_url, 'page_id', $this->integer(11)->comment('cms_page foreign key'));

neon/cms/migrations/m170107_201653_cms_create_cms_static_content_table_again.php

Indentation +6 added lines, -6 removed lines

@@ -4,8 +4,8 @@ 
 
 class m170107_201653_cms_create_cms_static_content_table_again extends Migration
 {
-    public function safeUp()
-    {
+	public function safeUp()
+	{
 		$cms_static = 'cms_static_content';
 
 		// recreate the table with the new structure
@@ -46,10 +46,10 @@ 
 //				}
 //			}
 //		} while ($count < $total);
-    }
+	}
 
-    public function safeDown()
-    {
+	public function safeDown()
+	{
 		$this->dropTable('cms_static_content');
-    }
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 
 		// recreate the table with the new structure
 		$this->execute("DROP TABLE IF EXISTS $cms_static");
-		$sql =<<<EOQ
+		$sql = <<<EOQ
 CREATE TABLE IF NOT EXISTS `cms_static_content` (
   `key` varchar(100) NOT NULL COMMENT 'a key to access the content' COLLATE utf8mb4_unicode_ci,
   `page_id` INT UNSIGNED NULL COMMENT 'if provided, restrict content to the associated page',

neon/cms/migrations/m170112_201653_cms_static_content_page_ids_to_uuid64.php

Indentation +8 added lines, -8 removed lines

@@ -7,23 +7,23 @@
 
 class m170112_201653_cms_static_content_page_ids_to_uuid64 extends Migration
 {
-    public function safeUp()
-    {
-    	$cms_static_content = CmsStaticContent::tableName();
+	public function safeUp()
+	{
+		$cms_static_content = CmsStaticContent::tableName();
 		$cms_page = CmsPage::tableName();
 
 		// use a nice id
-    	$this->alterColumn($cms_static_content, 'page_id', $this->char(22)->null()->comment('A uuid using base 64'));
+		$this->alterColumn($cms_static_content, 'page_id', $this->char(22)->null()->comment('A uuid using base 64'));
 		// update with nice_id's
 		$this->execute("UPDATE $cms_static_content JOIN $cms_page ON $cms_page.old_id = $cms_static_content.page_id SET $cms_static_content.page_id = $cms_page.id");
-    }
+	}
 
-    public function safeDown()
-    {
+	public function safeDown()
+	{
 		$cms_static_content = CmsStaticContent::tableName();
 		$cms_page = CmsPage::tableName();
 
 		$this->execute("UPDATE $cms_static_content JOIN $cms_page ON $cms_page.id = $cms_static_content.page_id SET $cms_static_content.page_id = $cms_page.old_id");
 		$this->alterColumn($cms_static_content, 'page_id', $this->integer(11)->null());
-    }
+	}
 }
\ No newline at end of file

neon/cms/migrations/m170121_150942_cms_delete_ddt_cms_static.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 		// created by an earlier cms migration so always appears if the
 		// migration creation order is not adhered to.
 		$connection = neon()->db;
-$sql =<<<EOQ
+$sql = <<<EOQ
 SET foreign_key_checks = 0;
 DROP TABLE IF EXISTS `ddt_cms_static_content`;
 DELETE FROM `dds_member` WHERE `class_type`='cms_static_content';

neon/cms/migrations/m170112_211700_change_dds_page_ids_to_uuid64.php

Indentation +10 added lines, -10 removed lines

@@ -7,12 +7,12 @@ 
 
 class m170112_211700_change_dds_page_ids_to_uuid64 extends Migration
 {
-    public function safeUp()
-    {
-    	$cms_page = CmsPage::tableName();
-    	// Step 1
+	public function safeUp()
+	{
+		$cms_page = CmsPage::tableName();
+		// Step 1
 		// ------
-    	// loop through all members in the DDS and find all members with a definition using
+		// loop through all members in the DDS and find all members with a definition using
 		// {class: neon\\cms\\form\\fields\\PageSelector}
 		// or where the definition uses a `dataMapKey` of `pages` and a `dataMapProvider` of `cms`
 		$members = neon()->db->query()->select('*')->from('dds_member')->where('definition LIKE \'{"class":"neon__cms__form__fields__PageSelector%\'')->all();
@@ -47,10 +47,10 @@ 
 			}
 		}
 		$this->dropColumn($cms_page, 'old_id');
-    }
+	}
 
-    public function safeDown()
-    {
-    	echo "There is no coming back from this!\n";
-    }
+	public function safeDown()
+	{
+		echo "There is no coming back from this!\n";
+	}
 }
\ No newline at end of file

Spacing +3 added lines, -3 removed lines

@@ -27,14 +27,14 @@
 		// - note the `member_ref` as the $column in the ddt table  - this column will hold the previous integer based page id
 		//   which will match the cms_page.old_id column
 		foreach ($members as $member) {
-			$class_type =  $member['class_type'];
-			$ddt_table_name =  'ddt_' . $class_type;
+			$class_type = $member['class_type'];
+			$ddt_table_name = 'ddt_'.$class_type;
 			$column = $member['member_ref'];
 
 			// Step 3
 			// ------
 			// Alter the $column in $ddt_table_name to be a uuid64 (char(22))
-			$exists = $this->db->createCommand("SELECT 1 FROM INFORMATION_SCHEMA.COLUMNS WHERE `table_name` = '$ddt_table_name' AND `table_schema` = '". env('DB_NAME')."' AND column_name = '$column'")->queryScalar();
+			$exists = $this->db->createCommand("SELECT 1 FROM INFORMATION_SCHEMA.COLUMNS WHERE `table_name` = '$ddt_table_name' AND `table_schema` = '".env('DB_NAME')."' AND column_name = '$column'")->queryScalar();
 			if ($exists) {
 				$this->alterColumn($ddt_table_name, $column, $this->char(22)->comment("A uuid using base 64"));
 				// Update the $column value in every row in the $ddt_table_name by matching it to the cms_page.old_id

neon/cms/migrations/m161130_180814_cms_additional_page_fields.php

Indentation +6 added lines, -6 removed lines

@@ -5,20 +5,20 @@
 class m161130_180814_cms_additional_page_fields extends Migration
 {
 
-    public function safeUp()
-    {
+	public function safeUp()
+	{
 		$this->addColumn('cms_page', 'keywords', "VARCHAR(1000) NULL COMMENT 'Keywords for this page'");
 		$this->addColumn('cms_page', 'description', "VARCHAR(1000) NULL COMMENT 'Description of this page'");
 		$this->addColumn('cms_page', 'languages', "VARCHAR(1000) NULL COMMENT 'Supported languages on this page'");
 		$this->addColumn('cms_page', 'page_params', "VARCHAR(10000) NULL COMMENT 'Additional parameters for this page which are interpreted directly within the templates'");
-    }
+	}
 
-    public function safeDown()
-    {
+	public function safeDown()
+	{
 		$this->dropColumn('cms_page', 'keywords');
 		$this->dropColumn('cms_page', 'description');
 		$this->dropColumn('cms_page', 'languages');
 		$this->dropColumn('cms_page', 'page_params');
-    }
+	}
 
 }

neon/cms/controllers/EditorController.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
 	 * @param null $id
 	 * @return string
 	 */
-	public function actionIndex($id=null)
+	public function actionIndex($id = null)
 	{
 		(new \neon\core\form\fields\Ide('ide'))->registerScripts(neon()->view);
 		$pageResults = CmsPage::find()->asArray()->orderBy('name')->all();
@@ -63,7 +63,7 @@ 
 		if ($pageForm->processRequest()) {
 			if (!$pageForm->save($errors)) {
 				// something went wrong
-				throw new \Exception( 'Something went wrong ' . print_r($errors, true));
+				throw new \Exception('Something went wrong '.print_r($errors, true));
 			}
 			$data = $pageForm->getData();
 			return $this->redirect(['/cms/editor/index', 'id' => $data['nice_id']]);
@@ -78,7 +78,7 @@ 
 	public function actionPageUpdate()
 	{
 		$data = neon()->request->post();
-		abort_if(! isset($data['CmsPage']['id']), 400, 'A "CmsPage[id]" parameter must be supplied');
+		abort_if(!isset($data['CmsPage']['id']), 400, 'A "CmsPage[id]" parameter must be supplied');
 		$cmsPage = CmsPage::findOne($data['CmsPage']['id']);
 		$cmsPage->load(neon()->request->post());
 		$cmsPage->save();

neon/cms/controllers/RedirectsController.php

Spacing +2 added lines, -2 removed lines

@@ -44,9 +44,9 @@
 	public function actionDelete()
 	{
 		if (!request()->isPost) abort(400, 'Must be a post method');
-		$url = request()->postRequired('url', );
+		$url = request()->postRequired('url',);
 		$redirect = CmsUrl::find()->where(['not', ['redirect' => null]])->andWhere(['url' => $url])->one();
-		if ($redirect===null) abort(404);
+		if ($redirect === null) abort(404);
 		return $redirect->delete();
 	}
 }

Braces +9 added lines, -3 removed lines

@@ -23,7 +23,9 @@ 
 	 */
 	public function actionAdd()
 	{
-		if (!request()->isPost) abort(400, 'Must be a post method');
+		if (!request()->isPost) {
+			abort(400, 'Must be a post method');
+		}
 
 		$req = request()->validateData(['url' => null, 'redirect' => null], [
 			[['url', 'redirect'], 'required'],
@@ -43,10 +45,14 @@ 
 
 	public function actionDelete()
 	{
-		if (!request()->isPost) abort(400, 'Must be a post method');
+		if (!request()->isPost) {
+			abort(400, 'Must be a post method');
+		}
 		$url = request()->postRequired('url', );
 		$redirect = CmsUrl::find()->where(['not', ['redirect' => null]])->andWhere(['url' => $url])->one();
-		if ($redirect===null) abort(404);
+		if ($redirect===null) {
+			abort(404);
+		}
 		return $redirect->delete();
 	}
 }