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

pulse plugin incompatibility #123

Open
SYoung3000 opened this issue Jul 8, 2024 · 1 comment
Open

pulse plugin incompatibility #123

SYoung3000 opened this issue Jul 8, 2024 · 1 comment

Comments

@SYoung3000
Copy link

Subject of the issue/enhancement/features

When using the pulse plugin alongside softcourse format, an error is thrown when trying to save the activity settings.. It appears the plugins interactor poorly. As I am unsure where the error resides, I've raised to both parties on their git.

Steps to reproduce

Create a course, set the course to softcourse. Add a Pulse activity. An error will be thrown.

The same error can also be triggered by setting another format, for example topics, then adding a pulse activity, and then switching the course format to softcourse.

Expected behaviour

the pulse activity should be added to the course

Actual behaviour

The following error is thrown

Stack trace:
line 157 of /course/format/softcourse/classes/output/courseformat/content/section.php: Error thrown
line 161 of /course/format/classes/output/local/content.php: call to format_softcourse\output\courseformat\content\section->export_for_template()
line 86 of /course/format/classes/output/local/content.php: call to core_courseformat\output\local\content->export_sections()
line 101 of /course/format/softcourse/classes/output/renderer.php: call to core_courseformat\output\local\content->export_for_template()
line 104 of /course/format/classes/output/section_renderer.php: call to format_softcourse\output\renderer->render_content()
line 49 of /course/format/softcourse/format.php: call to core_courseformat\output\section_renderer->render()
line 276 of /course/view.php: call to require()

@SYoung3000 SYoung3000 changed the title Bug report Pimenko pulse plugin incompatibility Jul 8, 2024
@CrymSonir
Copy link
Contributor

Hello,

I confirm that this issue is likely on the SoftCourse side. It's a problem we encountered and addressed for the "label" activity, but it seems we need to find a more generic solution. Since Pulse operates similarly to the label, this activity triggers the same error.

We currently lack the free time to look into and address this type of problem, so I can't guarantee that we'll handle it in the coming days. A "workaround" I can suggest would be either to avoid having the Pulse activity first in the order of section activities, or to "hide for student", which allows you to avoid the problem.

Best regards,

The Pimenko Tech Team

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

No branches or pull requests

2 participants