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

Question type fill in blanks or form change blanks marker causes bug in question #3485

Closed
roelvisje opened this issue Aug 27, 2020 · 2 comments
Closed

Question type fill in blanks or form change blanks marker causes bug in question #3485

roelvisje opened this issue Aug 27, 2020 · 2 comments
Assignees
@ywarnier
Labels
Enhancement
Milestone
1.11.14

Comments

@roelvisje
Copy link

roelvisje commented Aug 27, 2020 •
edited

When you create a fill in blanks type of question you use [...] to create a part where a student can answer a question.
Today i was showing some teachers this feature and also wanted to show you can change the [...] part into something different
but when you use that option, the question bugs that the already used [...] is gone…
the refresh terms button is also not working.
You cannot change back to the original setting since select a blanks marker is not working anymore.

before change
fill in blanks or form

after change
bugged

/edit/

ah, i see it is not a bug but sort a mistake in my case: the brackets also need to change form.
since i changed from [ -> (.. now and it works again… the text above didn't change so i thought it still needed the [] instead of ().. maybe that can be feature/enhancement then? the below (please define also still says [...]..
carlangas159 added a commit to carlangas159/chamilo-lms that referenced this issue Sep 15, 2020
@carlangas159
Quiz: Replace [...] for select typo - refs chamilo#3485
91f1dcc
ywarnier pushed a commit that referenced this issue Nov 12, 2020
@carlangas159
Quiz: Replace [...] by selected marker in feedback message - refs #3485
4c57f12
carlangas159 added a commit to carlangas159/chamilo-lms that referenced this issue Nov 13, 2020
@carlangas159
Merge remote-tracking branch 'chamilo/1.11.x' into BT_17891
e19024a 
* chamilo/1.11.x: (113 commits)
  Course copy: Dont copy test category in a session BT#17832
  Admin: Customize link to documentation to link to existing language if it exists.
  Minor - Use correct id, insert as int
  Admin: Add link to users guides on admin homepage
  Update language terms
  Minor - add log, fix typo
  Update language terms
  Portfolio: Revert lang term renaming - refs chamilo#3626
  Portfolio: Add help text to portfolio fields in user profile - refs chamilo#3636
  Update language terms
  Scripts: Add script to move a session from URL X to URL Y BT#18023
  Minor: Fix language term name - refs BT#17785
  LP: Add class in report filter BT#17370
  Minor: Fix language term name - refs BT#17785
  Minor - format code
  Minor - use api_get_user_id()
  Quiz: Replace [...] by selected marker in feedback message - refs chamilo#3485
  Survey: Doodle type: Set UTC datetime at save's moment of end and start date - refs chamilo#3499
  LDAP: Improved ldap (OpenLDAP vs ActiveDirectory) integration by providing better documented examples
  Documents: Fix issue moving folders in group spaces - Replace the $group_dir that may contain __0__1 which differs from the actual directory - refs chamilo#3505
  ...

# Conflicts:
#	main/install/configuration.dist.php
carlangas159 added a commit to carlangas159/chamilo-lms that referenced this issue Nov 19, 2020
@carlangas159
Merge remote-tracking branch 'chamilo/1.11.x' into BT_17931
e72c1fb 
* chamilo/1.11.x: (173 commits)
  Course copy: Dont copy test category in a session BT#17832
  Admin: Customize link to documentation to link to existing language if it exists.
  Minor - Use correct id, insert as int
  Admin: Add link to users guides on admin homepage
  Update language terms
  Minor - add log, fix typo
  Update language terms
  Portfolio: Revert lang term renaming - refs chamilo#3626
  Portfolio: Add help text to portfolio fields in user profile - refs chamilo#3636
  Update language terms
  Scripts: Add script to move a session from URL X to URL Y BT#18023
  Minor: Fix language term name - refs BT#17785
  LP: Add class in report filter BT#17370
  Minor: Fix language term name - refs BT#17785
  Minor - format code
  Minor - use api_get_user_id()
  Quiz: Replace [...] by selected marker in feedback message - refs chamilo#3485
  Survey: Doodle type: Set UTC datetime at save's moment of end and start date - refs chamilo#3499
  LDAP: Improved ldap (OpenLDAP vs ActiveDirectory) integration by providing better documented examples
  Documents: Fix issue moving folders in group spaces - Replace the $group_dir that may contain __0__1 which differs from the actual directory - refs chamilo#3505
  ...
@ywarnier
Copy link
Member

@roelvisje We could reproduce the issue and have hopefully fixed it through the commit 4c57f12
If you know how to apply a commit on your installation and can report back to us, we'd love that very much. Otherwise, this issue will be considered fixed in 5 days from now (Wednesday).

@ywarnier ywarnier added this to the 1.11.14 milestone Nov 20, 2020
@ywarnier ywarnier assigned ywarnier and unassigned carlangas159 Nov 20, 2020
@roelvisje
Copy link
Author

@ywarnier i can confirm it works and is fixed by the commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ywarnier ywarnier

Labels
Enhancement
Projects
None yet
Milestone
1.11.14
Development

No branches or pull requests
3 participants
@ywarnier @carlangas159 @roelvisje