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

添加卡券创建二维码接口 #608

Merged
merged 3 commits into from
Mar 8, 2017
Merged

添加卡券创建二维码接口 #608

merged 3 commits into from
Mar 8, 2017

Conversation

forecho
Copy link

@forecho forecho commented Mar 7, 2017

No description provided.

*
* @return string
*/
public static function getQRCodeUrl($ticket)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

其实不需要静态

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

静态为了方便单独调用

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@forecho 我们的所有模块在 app 里都已经单例了,实际上无所谓的,并不会重复创建对象,与静态调用差别不大

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

其实是为了不用传 $appId$appSecret 参数,我单独调用这个方法,没必要特意在去找这两个参数,而且这个方法不需要 AccessToken 可以单独使用。

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@overtrue 好吧,我改了。

@overtrue overtrue merged commit fef0dbb into w7corp:2.1 Mar 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants