We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
https://github.com/DQNEO/ethnam/blob/ethna_junction/class/Plugin/Abstract.php において、 配列の$this->configというプロパティが宣言されている。
$this->config
/** @protected array plugin configure */ protected $config;
これは問題点が3つある。
Plugin/Cachemanager*
ソースコードが無駄に複雑になってしまっているので、 class/Plugin/Abstract.phpには実装せず、 class/Plugin/Cachemanager.phpに実装する。
class/Plugin/Abstract.php
class/Plugin/Cachemanager.php
また、名称を$this->configではなく$this->optとする。
$this->opt
The text was updated successfully, but these errors were encountered:
issue #3 rename $this->config -> $this->opt
61bebac
issue #3 move _loadConfig from Abstract.php to Cachemanager.php
51da19e
完了した!! Plugin/Abstract.phpは廃止した。
Sorry, something went wrong.
No branches or pull requests
背景
https://github.com/DQNEO/ethnam/blob/ethna_junction/class/Plugin/Abstract.php
において、
配列の
$this->config
というプロパティが宣言されている。これは問題点が3つある。
Plugin/Cachemanager*
でしか使われていない。改善案
ソースコードが無駄に複雑になってしまっているので、
class/Plugin/Abstract.php
には実装せず、class/Plugin/Cachemanager.php
に実装する。また、名称を
$this->config
ではなく$this->opt
とする。The text was updated successfully, but these errors were encountered: