Skip to content
New issue

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

static メソッドの統一 #404

Closed
nanasess opened this issue Jun 18, 2020 · 1 comment · Fixed by #405
Closed

static メソッドの統一 #404

nanasess opened this issue Jun 18, 2020 · 1 comment · Fixed by #405
Milestone

Comments

@nanasess
Copy link
Contributor

nanasess commented Jun 18, 2020

  • PHP8対応 #397
  • SC_Helper_DB::sfGetBasisData() など、 static 宣言されていないのにも係わらず、 static アクセスされているメソッドがある
  • インスタンス生成 or static アクセスのどちらかに統一する
  • PHP5 では E_STRICT, PHP7 では E_DEPRECATED の警告が出ていた
  • PHP8 では廃止される予定(Fatal error になる)
  • static アクセスされているメソッドは、 基本は static 宣言を付与する
    • インスタンス経由で static メソッドをコールしても、エラーにはならないため
@nanasess nanasess mentioned this issue Jun 19, 2020
6 tasks
@okazy okazy added this to the 2.17.1 milestone Jun 23, 2020
@ryo-endo ryo-endo modified the milestones: 2.17.1, 2.17.2 Jun 29, 2020
@nanasess
Copy link
Contributor Author

修正方針

  • static 宣言を付与したメソッド内で $this にて呼び出しているメソッドは static:: で呼び出すよう修正する
    • new にて生成したインスタンス経由で呼び出した場合と振舞いを揃えるため
  • static 宣言を付与したメソッド内で XXX_Ex:: にて呼び出しているメソッドは修正しない

nanasess added a commit to nanasess/ec-cube2 that referenced this issue Dec 11, 2020
nanasess added a commit to nanasess/ec-cube2 that referenced this issue Dec 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants