php/yii2
1
0
Fork 0
mirror of https://github.com/yiisoft/yii2.git synced 2025-11-22 17:48:37 +08:00
Code Issues Packages Projects Releases Wiki Activity

added decision about closure params to design decisions

Browse Source 
close #6875
This commit is contained in:
Carsten Brandt
2015-01-15 20:15:54 +01:00
parent 9c287cb651
commit f07ccb0c86
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
docs/internals/design-decisions.md
View File

@@ -15,3 +15,6 @@ the core developers.
3. **[Adding new auth client support](https://github.com/yiisoft/yii2/issues/1652)** 3. **[Adding new auth client support](https://github.com/yiisoft/yii2/issues/1652)**
For better maintenability, we will not add any additional auth clients to the core extension. They should be done For better maintenability, we will not add any additional auth clients to the core extension. They should be done
in terms of user extensions. in terms of user extensions.
4. **When using closures** it is recommended to **include all passed parameters** in the signature even if not all of them are
used. This way modifying or copying code is easier because all information is directly visible and it is not necessary to
look up which params are actually available in the documentation. ([#6584](https://github.com/yiisoft/yii2/pull/6584), [#6875](https://github.com/yiisoft/yii2/issues/6875))