fix: Lib/CleanUpLockFileのメソッドに切り替え

Console/Command/CleanUpShell.php

@@ -10,8 +10,8 @@
 
 App::uses('Current', 'NetCommons.Utility');
 App::uses('CleanUp', 'CleanUp.Model');
-App::uses('CleanUpExec', 'CleanUp.Lib');
 App::uses('CleanUpLog', 'CleanUp.Lib');
+App::uses('CleanUpLockFile', 'CleanUp.Lib');
 
 /**
  * CleanUp Shell
@@ -94,7 +94,7 @@ public function clean_up() {
  */
 	public function unlock() {
 		// ロックファイルの削除
-		CleanUpExec::deleteLockFileAndSetupLog();
+		CleanUpLockFile::deleteLockFileAndSetupLog();
 	}
 
 /**

Controller/CleanUpController.php

@@ -11,6 +11,7 @@
 App::uses('CleanUpAppController', 'CleanUp.Controller');
 App::uses('CleanUp', 'CleanUp.Model');
 App::uses('CleanUpExec', 'CleanUp.Lib');
+App::uses('CleanUpLockFile', 'CleanUp.Lib');
 App::uses('Folder', 'Utility');
 
 /**
@@ -83,8 +84,8 @@ public function delete() {
 		$this->set('cleanUpLog', $cleanUpLog);
 
 		// ロックファイル関係
-		$this->set('isLockFile', CleanUpExec::isLockFile());
-		$this->set('cleanUpStart', CleanUpExec::readLockFile());
+		$this->set('isLockFile', CleanUpLockFile::isLockFile());
+		$this->set('cleanUpStart', CleanUpLockFile::readLockFile());
 	}
 
 /**
@@ -107,8 +108,8 @@ public function clean_up_log() {
 		$this->set('cleanUpLog', $cleanUpLog);
 
 		//		// ロックファイル関係
-		//		$this->set('isLockFile', CleanUpExec::isLockFile());
-		//		$this->set('cleanUpStart', CleanUpExec::readLockFile());
+		//		$this->set('isLockFile', CleanUpLockFile::isLockFile());
+		//		$this->set('cleanUpStart', CleanUpLockFile::readLockFile());
 	}
 
 /**
@@ -135,7 +136,7 @@ public function unlock() {
 		}
 
 		// ロックファイルの削除
-		if (CleanUpExec::deleteLockFileAndSetupLog()) {
+		if (CleanUpLockFile::deleteLockFileAndSetupLog()) {
 			// メッセージ
 			$this->NetCommons->setFlashNotification(
 				__d('clean_up', 'Lock file was deleted.'), array('class' => 'success')

Lib/CleanUpExec.php

@@ -21,17 +21,6 @@
  */
 class CleanUpExec {
 
-/**
- * 自作のstatic initialize<br />
- * 当クラス最下部で呼び出してる
- *
- * @return void
- * @deprecated 廃止予定
- */
-	public static function initialize() {
-		CleanUpLockFile::initialize();
-	}
-
 /**
  * ファイルクリーンアップ呼び出し
  *
@@ -75,56 +64,6 @@ public static function isWindows() {
 		return false;
 	}
 
-/**
- * ロックファイルの作成と時刻の書き込み。バッチ実行開始時
- *
- * @return void
- * @deprecated
- */
-	public static function makeLockFile() {
-		CleanUpLockFile::makeLockFile();
-	}
-
-/**
- * ロックファイルの削除。バッチ終了時
- *
- * @return bool true:削除|false:ファイルなし
- * @deprecated
- */
-	public static function deleteLockFile() {
-		return CleanUpLockFile::deleteLockFile();
-	}
-
-/**
- * ロックファイルの削除とログ出力設定。ロックファイル強制削除用
- *
- * @return bool true:削除|false:ファイルなし
- * @deprecated
- */
-	public static function deleteLockFileAndSetupLog() {
-		return CleanUpLockFile::deleteLockFileAndSetupLog();
-	}
-
-/**
- * ロックファイルの存在確認
- *
- * @return bool true:ロックあり|false:ロックなし
- * @deprecated
- */
-	public static function isLockFile() {
-		return CleanUpLockFile::isLockFile();
-	}
-
-/**
- * ロックファイルの読み込み
- *
- * @return string ファイルクリーンアップ開始時刻
- * @deprecated
- */
-	public static function readLockFile() {
-		return CleanUpLockFile::readLockFile();
-	}
-
 /**
  * ログ開始時のタイムゾーン変更<br />
  * タイムゾーンを一時的に変更。ログ出力時間を例えば日本時間に。
@@ -181,6 +120,3 @@ public static function getLogFileNames() {
 	}
 
 }
-
-// 自作のstatic initialize
-CleanUpExec::initialize();

Lib/CleanUpLockFile.php

@@ -9,6 +9,7 @@
  */
 
 App::uses('NetCommonsTime', 'NetCommons.Utility');
+App::uses('CleanUpLog', 'CleanUp.Lib');
 
 /**
  * ファイルクリーンアップ ライブラリ
@@ -72,16 +73,16 @@ public static function deleteLockFile() {
  * @return bool true:削除|false:ファイルなし
  */
 	public static function deleteLockFileAndSetupLog() {
-		CleanUpExec::setupLog();
+		CleanUpLog::setupLog();
 		// ログ開始時のタイムゾーン変更
-		$timezone = CleanUpExec::startLogTimezone();
+		$timezone = CleanUpLog::startLogTimezone();
 
 		CakeLog::info(__d('clean_up',
 			'Start forcibly delete lock file processing.'), ['CleanUp']);
 		$isDelete = self::deleteLockFile();
 
 		// ログ終了時にタイムゾーン戻す
-		CleanUpExec::endLogTimezone($timezone);
+		CleanUpLog::endLogTimezone($timezone);
 		return $isDelete;
 	}
 

Model/CleanUp.php

@@ -12,6 +12,7 @@
 App::uses('NetCommonsUrl', 'NetCommons.Utility');
 App::uses('NetCommonsTime', 'NetCommons.Utility');
 App::uses('CleanUpExec', 'CleanUp.Lib');
+App::uses('CleanUpLockFile', 'CleanUp.Lib');
 
 /**
  * CleanUp Model
@@ -136,7 +137,7 @@ public function beforeValidate($options = array()) {
  */
 	public function isLockFile($check) {
 		// @codingStandardsIgnoreEnd
-		return !CleanUpExec::isLockFile();
+		return !CleanUpLockFile::isLockFile();
 	}
 
 /**
@@ -262,7 +263,7 @@ public function fileCleanUp($data) {
 		CakeLog::info(__d('clean_up', 'Start cleanup process.'), ['CleanUp']);
 
 		// 複数起動防止ロック
-		CleanUpExec::makeLockFile();
+		CleanUpLockFile::makeLockFile();
 
 		// ファイルクリーンアップ対象のプラグイン設定を取得
 		$cleanUps = $this->getCleanUpsAndPlugin($data);
@@ -313,14 +314,14 @@ public function fileCleanUp($data) {
 
 		} catch (Exception $ex) {
 			// ロック解除
-			CleanUpExec::deleteLockFile();
+			CleanUpLockFile::deleteLockFile();
 			// タイムゾーンを元に戻す
 			CleanUpExec::endLogTimezone($timezone);
 			//トランザクションRollback
 			$this->rollback($ex);
 		}
 		// ロック解除
-		CleanUpExec::deleteLockFile();
+		CleanUpLockFile::deleteLockFile();
 		CakeLog::info(__d('clean_up', 'Cleanup processing is completed.'), ['CleanUp']);
 		// タイムゾーンを元に戻す
 		CleanUpExec::endLogTimezone($timezone);

Test/Case/Console/Command/CleanUpShell/UnlockTest.php

@@ -9,6 +9,7 @@
  */
 
 App::uses('CleanUpConsoleTestCase', 'CleanUp.TestSuite');
+App::uses('CleanUpLockFile', 'CleanUp.Lib');
 
 /**
  * CleanUpShell::unlock()のテスト
@@ -49,7 +50,7 @@ public function testUnlock() {
 		$this->$shell = $this->loadShell($shell);
 
 		//データ生成
-		CleanUpExec::makeLockFile();
+		CleanUpLockFile::makeLockFile();
 
 		//$this->$shell->expects($this->at(0))->method('out')
 		//	->with('ここに出力内容を書く');

Test/Case/Controller/CleanUpController/DeleteTest.php

@@ -10,8 +10,8 @@
 
 App::uses('CleanUpControllerTestCase', 'CleanUp.TestSuite');
 App::uses('CleanUpTestUtil', 'CleanUp.Test/Case');
-App::uses('CleanUpExec', 'CleanUp.Lib');
 App::uses('CleanUpLog', 'CleanUp.Lib');
+App::uses('CleanUpLockFile', 'CleanUp.Lib');
 
 /**
  * CleanUpController::delete()のテスト
@@ -95,7 +95,7 @@ public function testDeleteGet() {
  */
 	public function testDeleteGetIsLock() {
 		// ロックファイル作成
-		CleanUpExec::makeLockFile();
+		CleanUpLockFile::makeLockFile();
 
 		//テスト実行
 		$this->_testGetAction(

Test/Case/Controller/CleanUpController/UnlockTest.php

@@ -10,6 +10,7 @@
 
 App::uses('CleanUpControllerTestCase', 'CleanUp.TestSuite');
 App::uses('CleanUpTestUtil', 'CleanUp.Test/Case');
+App::uses('CleanUpLockFile', 'CleanUp.Lib');
 
 /**
  * CleanUpController::unlock()のテスト
@@ -75,7 +76,7 @@ public function tearDown() {
  */
 	public function testUnlockGetLocked() {
 		// ロックファイル作成
-		CleanUpExec::makeLockFile();
+		CleanUpLockFile::makeLockFile();
 
 		//テスト実行
 		$this->_testGetAction(

Test/Case/Lib/CleanUpExec/DeleteLockFileAndSetupLogTest.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * CleanUpExec::deleteLockFileAndSetupLog()のテスト
+ * CleanUpLockFile::deleteLockFileAndSetupLog()のテスト
  *
  * @author Mitsuru Mutaguchi <mutaguchi@opensource-workshop.jp>
  * @link http://www.netcommons.org NetCommons Project
@@ -9,9 +9,10 @@
  */
 
 App::uses('CleanUpCakeTestCase', 'CleanUp.TestSuite');
+App::uses('CleanUpLockFile', 'CleanUp.Lib');
 
 /**
- * CleanUpExec::deleteLockFileAndSetupLog()のテスト
+ * CleanUpLockFile::deleteLockFileAndSetupLog()のテスト
  *
  * @author Mitsuru Mutaguchi <mutaguchi@opensource-workshop.jp>
  * @package NetCommons\CleanUp\Test\Case\Utility\CleanUpExec
@@ -32,11 +33,11 @@ class CleanUpUtilityCleanUpUtilityDeleteLockFileAndSetupLogTest extends CleanUpC
  */
 	public function testDeleteLockFileAndSetupLog() {
 		//データ生成
-		CleanUpExec::makeLockFile();
+		CleanUpLockFile::makeLockFile();
 
 		//テスト実施
 		//チェック
-		$this->assertTrue(CleanUpExec::deleteLockFileAndSetupLog(), 'ロックファイルが削除できtrueの想定');
+		$this->assertTrue(CleanUpLockFile::deleteLockFileAndSetupLog(), 'ロックファイルが削除できtrueの想定');
 	}
 
 }

Test/Case/Lib/CleanUpExec/DeleteLockFileTest.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * CleanUpExec::deleteLockFile()のテスト
+ * CleanUpLockFile::deleteLockFile()のテスト
  *
  * @author Mitsuru Mutaguchi <mutaguchi@opensource-workshop.jp>
  * @link http://www.netcommons.org NetCommons Project
@@ -9,9 +9,10 @@
  */
 
 App::uses('CleanUpCakeTestCase', 'CleanUp.TestSuite');
+App::uses('CleanUpLockFile', 'CleanUp.Lib');
 
 /**
- * CleanUpExec::deleteLockFile()のテスト
+ * CleanUpLockFile::deleteLockFile()のテスト
  *
  * @author Mitsuru Mutaguchi <mutaguchi@opensource-workshop.jp>
  * @package NetCommons\CleanUp\Test\Case\Utility\CleanUpExec
@@ -32,11 +33,11 @@ class CleanUpUtilityCleanUpUtilityDeleteLockFileTest extends CleanUpCakeTestCase
  */
 	public function testDeleteLockFile() {
 		//データ生成
-		CleanUpExec::makeLockFile();
+		CleanUpLockFile::makeLockFile();
 
 		//テスト実施
 		//チェック
-		$this->assertTrue(CleanUpExec::deleteLockFile(), 'ロックファイルが削除できtrueの想定');
+		$this->assertTrue(CleanUpLockFile::deleteLockFile(), 'ロックファイルが削除できtrueの想定');
 	}
 
 /**
@@ -47,7 +48,7 @@ public function testDeleteLockFile() {
 	public function testDeleteLockFileCannotFile() {
 		//テスト実施
 		//チェック
-		$this->assertFalse(CleanUpExec::deleteLockFile(), 'ロックファイルがなくfalseの想定');
+		$this->assertFalse(CleanUpLockFile::deleteLockFile(), 'ロックファイルがなくfalseの想定');
 	}
 
 }

Test/Case/Lib/CleanUpExec/IntializeTest.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * CleanUpExec::initialize()のテスト
+ * CleanUpLockFile::initialize()のテスト
  *
  * @author Mitsuru Mutaguchi <mutaguchi@opensource-workshop.jp>
  * @link http://www.netcommons.org NetCommons Project
@@ -9,10 +9,10 @@
  */
 
 App::uses('CleanUpCakeTestCase', 'CleanUp.TestSuite');
-App::uses('CleanUpExec', 'CleanUp.Lib');
+App::uses('CleanUpLockFile', 'CleanUp.Lib');
 
 /**
- * CleanUpExec::initialize()のテスト
+ * CleanUpLockFile::initialize()のテスト
  *
  * @author Mitsuru Mutaguchi <mutaguchi@opensource-workshop.jp>
  * @package NetCommons\CleanUp\Test\Case\Utility\CleanUpExec
@@ -34,7 +34,7 @@ class CleanUpUtilityCleanUpUtilityiIntializeTest extends CleanUpCakeTestCase {
 	public function testInitialize() {
 		//テスト実施
 		// CakeLog::initialize()を実行してるだけなので、ロジックが通る事を確認
-		CleanUpExec::initialize();
+		CleanUpLockFile::initialize();
 
 		//チェック
 		$this->assertTrue(true);