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

Possible caching issue #6

Open
mackensen opened this issue Apr 3, 2017 · 4 comments
Open

Possible caching issue #6

mackensen opened this issue Apr 3, 2017 · 4 comments
Assignees
Labels

Comments

@mackensen
Copy link
Contributor

mackensen commented Apr 3, 2017

Stack trace observed when the deletion of 900+ courses in 60 subcategories was in progress:

Debug info: Argument 1 passed to course_in_list::__construct() must be an instance of stdClass, null given, called in [dirroot]/lib/coursecatlib.php on line 1508 and defined
Error code: codingerror

Trace:

Stack trace: 
line 426 of /lib/setuplib.php: coding_exception thrown
line 2788 of /lib/coursecatlib.php: call to default_error_handler()
line 1508 of /lib/coursecatlib.php: call to course_in_list->__construct()
line 524 of /course/classes/management_renderer.php: call to coursecat->get_courses()
line 498 of /course/management.php: call to core_course_management_renderer->course_listing()

This was triggered by viewing a subcategory in the category management page. My guess is that it expected courses which no longer exist. After a few minutes the category page can be refreshed and loads without error (and no courses).

@mackensen mackensen added the bug label Apr 3, 2017
@mackensen mackensen self-assigned this Apr 3, 2017
mackensen added a commit that referenced this issue Apr 3, 2017
@mackensen
Copy link
Contributor Author

I'm still observing the problem; I'm unsure if this is a core bug or if it points to a flaw in the deletion procedure the tool is using.

@kwiliarty
Copy link

I also see what I assume is the same problem. It's not a big deal, but I'm wondering if you believe 82e0887 addresses the issue.

@mackensen
Copy link
Contributor Author

I didn't merge it so I'm thinking that it didn't resolve it.

@kwiliarty
Copy link

Okay. It's not a major problem, and as you point out it eventually resolves itself. Thanks for this helpful tool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants