| @@ -326,7 +326,6 @@ | ||
| 326 | 326 | |
| 327 | 327 | /** | 
| 328 | 328 | * @param $row | 
| 329 | - * @param null $callback | |
| 330 | 329 | */ | 
| 331 | 330 | public function fputcsv($row) | 
| 332 | 331 |      { | 
| @@ -57,7 +57,7 @@ | ||
| 57 | 57 | /** | 
| 58 | 58 | * Constructor. | 
| 59 | 59 | * | 
| 60 | - * @param Query|QueryBuilder $query A Doctrine ORM query or query builder. | |
| 60 | + * @param Query $query A Doctrine ORM query or query builder. | |
| 61 | 61 | * @param boolean $fetchJoinCollection Whether the query joins a collection (true by default). | 
| 62 | 62 | */ | 
| 63 | 63 | public function __construct($query, $fetchJoinCollection = true) | 
| @@ -53,6 +53,9 @@ | ||
| 53 | 53 | $this->params = $params; | 
| 54 | 54 | } | 
| 55 | 55 | |
| 56 | + /** | |
| 57 | + * @param Expr\Comparison $expr | |
| 58 | + */ | |
| 56 | 59 | private static function newWhereClause($expr, $x, $y) | 
| 57 | 60 |      { | 
| 58 | 61 |          if ($y) { | 
| @@ -16,7 +16,6 @@ | ||
| 16 | 16 | * This method should only be used to configure services and parameters. | 
| 17 | 17 | * It should not get services. | 
| 18 | 18 | * | 
| 19 | - * @param Container $pimple A container instance | |
| 20 | 19 | */ | 
| 21 | 20 | public function register(Container $app) | 
| 22 | 21 |      { | 
| @@ -33,7 +33,6 @@ | ||
| 33 | 33 | use Eccube\Entity\Plugin; | 
| 34 | 34 | use Eccube\Entity\PluginEventHandler; | 
| 35 | 35 | use Eccube\Exception\PluginException; | 
| 36 | -use Eccube\Plugin\ConfigManager; | |
| 37 | 36 | use Eccube\Plugin\ConfigManager as PluginConfigManager; | 
| 38 | 37 | use Eccube\Repository\PluginEventHandlerRepository; | 
| 39 | 38 | use Eccube\Repository\PluginRepository; | 
| @@ -147,7 +147,7 @@ discard block | ||
| 147 | 147 | * @param string $path path to tar.gz/zip plugin file | 
| 148 | 148 | * @param int $source | 
| 149 | 149 | * | 
| 150 | - * @return mixed | |
| 150 | + * @return boolean | |
| 151 | 151 | * | 
| 152 | 152 | * @throws PluginException | 
| 153 | 153 | * @throws \Exception | 
| @@ -274,8 +274,8 @@ discard block | ||
| 274 | 274 | } | 
| 275 | 275 | |
| 276 | 276 | /** | 
| 277 | - * @param $archive | |
| 278 | - * @param $dir | |
| 277 | + * @param string $archive | |
| 278 | + * @param string $dir | |
| 279 | 279 | * | 
| 280 | 280 | * @throws PluginException | 
| 281 | 281 | */ | 
| @@ -344,6 +344,9 @@ discard block | ||
| 344 | 344 | } | 
| 345 | 345 | } | 
| 346 | 346 | |
| 347 | + /** | |
| 348 | + * @param string $yml | |
| 349 | + */ | |
| 347 | 350 | public function readYml($yml) | 
| 348 | 351 |      { | 
| 349 | 352 |          if (file_exists($yml)) { | 
| @@ -361,6 +364,9 @@ discard block | ||
| 361 | 364 | // ディレクトリ名などに使われれるので厳しめ | 
| 362 | 365 | } | 
| 363 | 366 | |
| 367 | + /** | |
| 368 | + * @param string $path | |
| 369 | + */ | |
| 364 | 370 | public function deleteFile($path) | 
| 365 | 371 |      { | 
| 366 | 372 | $f = new Filesystem(); | 
| @@ -381,7 +387,7 @@ discard block | ||
| 381 | 387 | } | 
| 382 | 388 | |
| 383 | 389 | /** | 
| 384 | - * @param $d | |
| 390 | + * @param string $d | |
| 385 | 391 | * | 
| 386 | 392 | * @throws PluginException | 
| 387 | 393 | */ | 
| @@ -456,7 +462,7 @@ discard block | ||
| 456 | 462 | |
| 457 | 463 | /** | 
| 458 | 464 | * @param $meta | 
| 459 | - * @param $method | |
| 465 | + * @param string $method | |
| 460 | 466 | */ | 
| 461 | 467 | public function callPluginManagerMethod($meta, $method) | 
| 462 | 468 |      { | 
| @@ -997,7 +1003,7 @@ discard block | ||
| 997 | 1003 | * [プラグインコード]/Resource/assets | 
| 998 | 1004 | * 配下に置かれているファイルが所定の位置へコピーされる | 
| 999 | 1005 | * | 
| 1000 | - * @param $pluginBaseDir | |
| 1006 | + * @param string $pluginBaseDir | |
| 1001 | 1007 | * @param $pluginCode | 
| 1002 | 1008 | */ | 
| 1003 | 1009 | public function copyAssets($pluginBaseDir, $pluginCode) | 
| @@ -1033,7 +1039,7 @@ discard block | ||
| 1033 | 1039 | * @param string $pluginVersion | 
| 1034 | 1040 | * @param string $remoteVersion | 
| 1035 | 1041 | * | 
| 1036 | - * @return mixed | |
| 1042 | + * @return boolean | |
| 1037 | 1043 | */ | 
| 1038 | 1044 | public function isUpdate($pluginVersion, $remoteVersion) | 
| 1039 | 1045 |      { | 
| @@ -27,7 +27,7 @@ | ||
| 27 | 27 | * | 
| 28 | 28 | * @param Application $app | 
| 29 | 29 | * | 
| 30 | - * @return \Symfony\Component\HttpFoundation\Response | |
| 30 | + * @return string | |
| 31 | 31 | */ | 
| 32 | 32 | public function testAdmin(Application $app) | 
| 33 | 33 |      { | 
| @@ -12,6 +12,9 @@ | ||
| 12 | 12 | |
| 13 | 13 | class DragAndDropBy extends WebDriverActions | 
| 14 | 14 |  { | 
| 15 | + /** | |
| 16 | + * @param \Facebook\WebDriver\Remote\RemoteWebElement $source | |
| 17 | + */ | |
| 15 | 18 | public function __construct(WebDriver $driver, $source, $x_offset, $y_offset) | 
| 16 | 19 |      { | 
| 17 | 20 | parent::__construct($driver); | 
| @@ -14,6 +14,9 @@ | ||
| 14 | 14 | */ | 
| 15 | 15 | private $timeout_in_second; | 
| 16 | 16 | |
| 17 | + /** | |
| 18 | + * @param integer $timeout_in_second | |
| 19 | + */ | |
| 17 | 20 | function __construct($timeout_in_second) | 
| 18 | 21 |      { | 
| 19 | 22 | $this->timeout_in_second = $timeout_in_second; | 
| @@ -48,12 +48,18 @@ discard block | ||
| 48 | 48 | return $this; | 
| 49 | 49 | } | 
| 50 | 50 | |
| 51 | + /** | |
| 52 | + * @param integer $rowNum | |
| 53 | + */ | |
| 51 | 54 | public function 一覧_編集($rowNum) | 
| 52 | 55 |      { | 
| 53 | 56 |          $this->tester->click("#search_form > div.c-contentsArea__cols > div > div > div.card.rounded.border-0.mb-4 > div > table > tbody > tr:nth-child(${rowNum}) > td:nth-child(2) > a"); | 
| 54 | 57 | return $this; | 
| 55 | 58 | } | 
| 56 | 59 | |
| 60 | + /** | |
| 61 | + * @param integer $rowNum | |
| 62 | + */ | |
| 57 | 63 | public function 一覧_削除($rowNum, $execute = true) | 
| 58 | 64 |      { | 
| 59 | 65 |          $this->tester->click("#search_form > div.c-contentsArea__cols > div > div > div.card.rounded.border-0.mb-4 > div > table > tbody > tr:nth-child(${rowNum}) > td.align-middle.pr-3 > div > div > a"); | 
| @@ -67,6 +73,9 @@ discard block | ||
| 67 | 73 | return $this; | 
| 68 | 74 | } | 
| 69 | 75 | |
| 76 | + /** | |
| 77 | + * @param integer $rowNum | |
| 78 | + */ | |
| 70 | 79 | public function 一覧_仮会員メール再送($rowNum) | 
| 71 | 80 |      { | 
| 72 | 81 |          $this->tester->click(['xpath' => "//*[@id='search_form']//div/table/tbody/tr[${rowNum}]/td[6]/div/div[1]/a"]); | 
| @@ -92,6 +101,9 @@ discard block | ||
| 92 | 101 |          $this->tester->click('#search_form > div.c-contentsArea__cols > div > div > div.row.justify-content-between.mb-2 > div.col-5.text-right > div:nth-child(2) > div > button:nth-child(2)'); | 
| 93 | 102 | } | 
| 94 | 103 | |
| 104 | + /** | |
| 105 | + * @param integer $rowNum | |
| 106 | + */ | |
| 95 | 107 | public function 一覧_会員ID($rowNum) | 
| 96 | 108 |      { | 
| 97 | 109 |          return $this->tester->grabTextFrom("#search_form > div.c-contentsArea__cols > div > div > div.card.rounded.border-0.mb-4 > div > table > tbody > tr:nth-child(${rowNum}) > td.align-middle.pl-3"); |