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

更改了默认的APP名字，测试时出现报错，应该是BUG #41

Closed
torome opened this issue Dec 30, 2015 · 4 comments
Closed

更改了默认的APP名字，测试时出现报错，应该是BUG #41

torome opened this issue Dec 30, 2015 · 4 comments

Comments

@torome
Copy link

torome commented Dec 30, 2015

FatalErrorException in SendReminderSms.php line 12: Class 'App\Jobs\Job' not found
@toplan
Copy link
Owner

toplan commented Dec 30, 2015

请问是否将app/Jobs文件夹下的Job抽象类的命名空间修改了吗？

@torome
Copy link
Author

torome commented Dec 30, 2015

是的，通过命令批量更新的，所以app/Jobs的命名空间也是更改了xxx/Jobs

@toplan
Copy link
Owner

toplan commented Dec 30, 2015

恩，了解了，我考虑的解决方案是，在config/laravel-sms.php中新加一项配置用来指定使用的队列任务job类，默认为App\Jobs\SendReminderSms，如果自己运行php artisan app:name或者自己想修改队列实现逻辑，则可以自己实现一个job类（Your\Namespace\YourJob），然后在配置中修改配置为Your\Namespace\YourJob。

当然如果有有更好方案，请告知 😄

toplan added a commit that referenced this issue Jan 3, 2016
@toplan
fix issue #41;use phpsms 1.2.0;
38ad041
@toplan
Copy link
Owner

toplan commented Jan 3, 2016

https://github.com/toplan/laravel-sms#2-队列自定义

@toplan toplan closed this as completed Jan 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@torome @toplan