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.

Sign up for GitHub

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

Jump to bottom

debug_backtrace() を使用している箇所の改善 #321

Merged
merged 17 commits into from
Nov 19, 2019
Merged

debug_backtrace() を使用している箇所の改善 #321

merged 17 commits into from
Nov 19, 2019

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Oct 21, 2019 •
edited
Loading

  • debug_backtrace() 関数で object が取得できない場合がある #320
    • debug_backtrace() で object が取得できないのを修正
    • 取得できない場合は次の配列を見にいくよう修正
    • static コールしていた関数で $this を参照していたのを修正
  • debug_backtrace() に適切なオプションを追加
    • スタックフレームの出力を最小限に修正
  • SC_Response クラスで static コールしていた関数に static 追加
  • SC_Display::reload() を非推奨に
  • テストケース追加

@nanasess
debug_backtrace() で object が取得できないのを修正
b466960 
- 取得できない場合は次の配列を見にいくよう修正
@coveralls
Copy link

coveralls commented Oct 21, 2019 •
edited
Loading

Coverage Status

Coverage increased (+0.9%) to 52.473% when pulling a4afe21 on nanasess:fix-actionexit-hookpoint into 0c3d0da on EC-CUBE:master.

@chihiro-adachi chihiro-adachi added this to the 2.17.0 milestone Oct 21, 2019
@nanasess
debug_backtrace() にオプションを付与してパフォーマンスを改善
559cbce 
- スタックフレームは2つまで
- args は省略
- 不要なメンバ変数の参照を削除
@nanasess
debug_backtrace() にオプションを付与してパフォーマンスを改善
6a33e46 
- スタックフレームは2つまで
- args は省略
@nanasess
debug_backtrace() にオプションを付与してパフォーマンスを改善
73a1003 
- スタックフレームは2つまで
- object と args は省略
@nanasess
debug_backtrace() にオプションを付与してパフォーマンスを改善
9389012 
- args は省略
@nanasess
テストケース追加
7842091
@kiy0taka kiy0taka changed the base branch from improve/php7 to master October 23, 2019 01:40
@nanasess nanasess changed the title [WIP] debug_backtrace() で object が取得できないのを修正 debug_backtrace() を使用している箇所の改善 Oct 25, 2019
@nanasess nanasess changed the title debug_backtrace() を使用している箇所の改善 [WIP]debug_backtrace() を使用している箇所の改善 Oct 27, 2019
@nanasess nanasess changed the title [WIP]debug_backtrace() を使用している箇所の改善 debug_backtrace() を使用している箇所の改善 Oct 28, 2019
@chihiro-adachi chihiro-adachi modified the milestones: 2.17.0, 2.17.1 Oct 29, 2019
@chihiro-adachi chihiro-adachi merged commit e51abe5 into EC-CUBE:master Nov 19, 2019
@nanasess nanasess mentioned this pull request Nov 20, 2019
Closed
@nanasess nanasess deleted the fix-actionexit-hookpoint branch October 3, 2022 08:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nobuhiko nobuhiko nobuhiko approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
2.17.1
Development

Successfully merging this pull request may close these issues.
4 participants
@nanasess @coveralls @nobuhiko @chihiro-adachi