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

C2 : Survey in LP : enable student to access #5713

Open
NicoDucou opened this issue Aug 7, 2024 · 4 comments
Open

C2 : Survey in LP : enable student to access #5713

NicoDucou opened this issue Aug 7, 2024 · 4 comments

Comments

@NicoDucou
Copy link
Member

In Chamilo Master you can include a survey in a LP.
When including a survey in a LP the student automatically gain access to the survey because it is included in the learning path no need to have an invitation.
In Chamilo Master on the 2024/08/07 we can include a survey in a LP but the student does not have access it is indicated "Protected document".
So it needs to be reimplemented in master to enable a student to access a survey in the LP. And if the course is included in a session then the student in the session that are in the LP need to be able to do the survey as well.

In Chamilo 1.11.x it was implemented with this option :

// Allow survey tool in learnpath
// ALTER TABLE c_survey_answer ADD COLUMN c_lp_item_id INT DEFAULT 0;
// Edit src/Chamilo/CourseBundle/Entity/CSurveyAnswer.php and add a '' character in front of 'ORM\Column(name="c_lp_item_id"
// ALTER TABLE c_survey_invitation ADD COLUMN c_lp_item_id int(11) DEFAULT 0;
// Edit src/Chamilo/CourseBundle/Entity/CSurveyInvitation.php and add a '' character in front of 'ORM\Column(name="c_lp_item_id"
//$_configuration['allow_survey_tool_in_lp'] = false;

@christianbeeznest christianbeeznest self-assigned this Aug 7, 2024
@christianbeeznest
Copy link
Contributor

Hi @NicoDucou ,

The required fields have been added to the entities for those tables, and the default setting is now to show surveys in lessons. Therefore, the allow_survey_tool_in_lp option is no longer used in Chamilo 2, and it has been removed in the migration file Version20240423101010:

@NicoDucou
Copy link
Member Author

Hi @christianbeeznest
Thank you for the indication.
Indeed the option "allow_survey_tool_in_lp" is not necessary anymore but the survey should be accessible for the student if it is in a LP. For the moment the student gets an indication of "Protected document".

The way it should work is if a survey is included in a LP then the user should have access to it no need to be invited. And if the course is included in a session then the student in the session that are in the LP need to be able to do the survey as well.

@NicoDucou
Copy link
Member Author

christianbeeznest added a commit to christianbeeznest/chamilo-lms that referenced this issue Aug 30, 2024
@christianbeeznest
Copy link
Contributor

Hi @NicoDucou ,

It is added in this PR #5766

Thanks for confirmation.

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

No branches or pull requests

2 participants