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

Add missing import PtpUploaderMessageStopJob #28

Merged
merged 1 commit into from
Jun 5, 2022

Conversation

LeFirstTimer
Copy link

@LeFirstTimer LeFirstTimer commented Jun 5, 2022

Trying to stop a job throws 500:
2022-06-05 18:19:54,717 ERROR django.request Internal Server Error: /job/240/stop
Traceback (most recent call last):
File "..<path>../django/core/handlers/exception.py", line 47, in inner
response = get_response(request)
File ""..<path>../django/core/handlers/base.py", line 181, in _get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File ""..<path>../django/contrib/auth/decorators.py", line 21, in _wrapped_view
return view_func(request, *args, **kwargs)
File ""..<path>../PtpUploader/web/views.py", line 344, in stop_job
MyGlobals.PtpUploader.add_message(PtpUploaderMessageStopJob(r_id))
NameError: name 'PtpUploaderMessageStopJob' is not defined

This bug was introduced when the linting changes were committed 5 days ago:
6ad3990

Trying to stop a job throws 500:
2022-06-05 18:19:54,717 ERROR django.request Internal Server Error: /job/240/stop
Traceback (most recent call last):
  File "..<path>../django/core/handlers/exception.py", line 47, in inner
    response = get_response(request)
  File ""..<path>../django/core/handlers/base.py", line 181, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File ""..<path>../django/contrib/auth/decorators.py", line 21, in _wrapped_view
    return view_func(request, *args, **kwargs)
  File ""..<path>../PtpUploader/web/views.py", line 344, in stop_job
    MyGlobals.PtpUploader.add_message(PtpUploaderMessageStopJob(r_id))
NameError: name 'PtpUploaderMessageStopJob' is not defined
@kannibalox kannibalox merged commit 4d86732 into kannibalox:main Jun 5, 2022
@kannibalox
Copy link
Owner

Thanks for catching that!

@LeFirstTimer LeFirstTimer deleted the patch-2 branch June 5, 2022 19:14
@LeFirstTimer LeFirstTimer restored the patch-2 branch June 5, 2022 19:14
@LeFirstTimer LeFirstTimer deleted the patch-2 branch June 5, 2022 19:15
@LeFirstTimer LeFirstTimer restored the patch-2 branch June 5, 2022 19:15
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