Skip to content

fix(survey): normalize survey_type to int to avoid strict type mismatch on PHP 8.3 #6841

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
Open
wants to merge 10,000 commits into
base: 1.11.x
Choose a base branch
from
Open

fix(survey): normalize survey_type to int to avoid strict type mismatch on PHP 8.3 #6841

wants to merge 10,000 commits into from

Conversation

Claudius21
Copy link

fix(survey): normalize survey_type to int to avoid strict type mismatch on PHP 8.3

Bildschirmfoto_20251001_170338-2

Fixes: Survey type unknown / Umfragetyp unbekannt

ywarnier and others added 30 commits September 9, 2025 18:36
@ywarnier
Merge branch 'master' of github.com:chamilo/chamilo-lms
08d403b
@ywarnier
Plugin: BuyCourse: Fix missing status in currency table - refs chamil…
344475b 
…o#2615
@christianbeeznest
Assignment: Correct deadline/end-date columns - refs chamilo#6721
e6b6597
@christianbeeznest
Merge pull request chamilo#6727 from christianbeeznest/chamiloGH-6721
1f09af7 
Assignment: Correct deadline/end-date columns - refs chamilo#6721
@ywarnier
Plugin: BuyCourse: Add entries for BTC and DOGE in currencies list an…
d961d0b 
…d update ISOcode column to 4 characters (prepare for incremental crypto support)
@ywarnier
Merge branch 'master' of github.com:chamilo/chamilo-lms
8203dd5
@christianbeeznest
Assignment: Remove “Sent date” (teacher) and hide “End date” (student…
6b9361c 
…) - refs chamilo#6721
@christianbeeznest
Merge pull request chamilo#6728 from christianbeeznest/chamiloGH-6721-2
efe5fb5 
Assignment: Remove “Sent date” (teacher) and hide “End date” (student) - refs chamilo#6721
@ywarnier
Plugin: Remove use of 'settings_current' as a literal table name in p…
11ba325 
…lugin.class.php
@ywarnier
Minor: Plugin: BuyCourse: Update documentation
e356c49
@ywarnier
Merge branch 'master' of github.com:chamilo/chamilo-lms
c9ec086
@christianbeeznest
Assignment: Allow student submissions until end date
66a020e
@christianbeeznest
Merge pull request chamilo#6729 from christianbeeznest/fixes-updates132
624aae6 
Assignment: Allow student submissions until end date
@christianbeeznest
Assignment: Resolve child-entity error on delete - refs chamilo#6134
72861dd
@christianbeeznest
Merge pull request chamilo#6730 from christianbeeznest/chamiloGH-6134-4
ea09e17 
Assignment: Resolve child-entity error on delete - refs chamilo#6134
@christianbeeznest
Course: Include optional tracking of users of a course who are not su…
dc14cca 
…bscribed - refs chamilo#5958
@christianbeeznest
Merge pull request chamilo#6734 from christianbeeznest/chamiloGH-5958-2
b0bc0a7 
Course: Include optional tracking of users of a course who are not subscribed - refs chamilo#5958
@NicoDucou
Language: add fr translation for specific functionnalities for Sessio…
45e07c1 
…n admins - refs BT#22437
@NicoDucou
Language: update vue translations
ad90ade
@NicoDucou
Language: add 1 missing translation in FR for specific functionnality…
9b6fb58 
… for Session admins - refs BT#22437
@NicoDucou
Language: fix translation variable - refs BT#22437
604c0b1
@NicoDucou
Language: fix translation variable - refs BT#22437
1d4c849
@christianbeeznest
Catalog: Improve list of extrafields in course catalog
0944caf
@christianbeeznest
Merge pull request chamilo#6736 from christianbeeznest/fixes-updates133
5a1e796 
Catalog: Improve list of extrafields in course catalog
@ywarnier
Language: Add missing terms
5342eb2
@christianbeeznest
Course: Add toggle to show/hide “(free)” non-enrolled users in tracki…
6f34b66 
…ng table - refs chamilo#5958
@christianbeeznest
Merge pull request chamilo#6737 from christianbeeznest/chamiloGH-5958-3
afcf3d1 
Course: Add toggle to show/hide “(free)” non-enrolled users in tracking table - refs chamilo#5958
@christianbeeznest
Course: Fixed and improve filter by extra fields from catalog - refs c…
5675e12 
…hamilo#6235
@christianbeeznest
Merge pull request chamilo#6740 from christianbeeznest/chamiloGH-6235-4
c2c3b5c 
Course: Fixed and improve filter by extra fields from catalog - refs chamilo#6235
@christianbeeznest
Course: Improve optional tracking of users of a course who are not su…
ea2dd28 
…bscribed - refs chamilo#5958
AngelFQC and others added 27 commits September 29, 2025 10:52
@AngelFQC
Fix install BBB plugin installation chamilo#2615
d41e1e1
@AngelFQC
Replacing legacy routes in templates chamilo#2615
ec03f37
@AngelFQC
Replacing legacy routes in templates chamilo#2615
77d3751
@AngelFQC
Plugin: BuyCourses: Fix course/session/service list chamilo#2615
8723128
@ywarnier
Display: Slightly improve display of admin report links and remove no…
709c10b 
…w unused method MySpace::getAdminActions()
@ywarnier
Merge branch 'master' of github.com:chamilo/chamilo-lms
cad266f
@ywarnier
Display: Fix translations for user fields in profile edition
c8834ac
@ywarnier
Script: Add support for 'display_text' fixture use for default extra …
46fd010 
…fields + better removal of quotes
@ywarnier
Minor: Fix typo in PHPDoc block
573a610
@AngelFQC
Plugin: BuyCourses: disable autoscape in templates chamilo#2615
010cd8e
@AngelFQC
Merge branch 'master' of github.com:chamilo/chamilo-lms
9c6c7e6
@ywarnier
Language: Fix language terms in course creation form
36e9b2d
@ywarnier
Merge branch 'master' of github.com:chamilo/chamilo-lms
087331d
@NicoDucou
Quiz: #fix message to be shown only if user is course trainer and fix…
f502ed6 
… french translation - refs BT#22897
@ywarnier
Language: Update Arabic translation (fix wrongly untranslated terms)
e2d6710
@ywarnier
Language: Update Arabic translation
b651eeb
@NicoDucou
Language: add missing variable - refs BT#22741
864b766
@AngelFQC
Update LoginTokenAuthenticator.php
5f5177e
@christianbeeznest
Internal: Update extrafield flags for 'my_terms','terms_ville','terms…
497f82f 
…_villedustage' not included in migration
@christianbeeznest
Merge pull request chamilo#6840 from christianbeeznest/fixes-updates147
3eb8eb9 
Internal: Update extrafield flags for 'my_terms','terms_ville','terms_villedustage' not included in migration
@ywarnier
Language: Add JSON translation file for brazilian Portuguese
ddec0d5
@ywarnier
Merge branch 'master' of github.com:chamilo/chamilo-lms
e9e739b
@christianbeeznest
Dropbox: Re-enable blogs tool - refs chamilo#6823
1e7f196
@ywarnier
Plugin: BuyCourses: Fix name field renamed to title - refs chamilo#2615
fa3ffda
@ywarnier
Merge branch 'master' of github.com:chamilo/chamilo-lms
5eec209
@NicoDucou
Survey: #fix remove c_id and question_id from queries because they do…
642e283 
… not exist anymore in the new structure - refs BT#22992
@Claudius21
fix(survey): normalize survey_type to int to avoid strict type mismat…
4d425f3 
…ch on PHP 8.3

fix(survey): normalize survey_type to int to avoid strict type mismatch on PHP 8.3
@ywarnier
Copy link
Member

ywarnier commented Oct 6, 2025

Hi @Claudius21
Based on your screenshot, you want this to apply to 1.11.34 of Chamilo, right? I'm asking because you sent your PR on the master branch, which is for Chamilo 2.0. If you wanted it to affect the next 1.11, could you please edit your PR to make it apply to chamilo-lms/1.11.x?

@Claudius21 Claudius21 changed the base branch from master to 1.11.x October 7, 2025 09:17
@Claudius21
Copy link
Author

Sorry, my bad.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Claudius21 @ywarnier @christianbeeznest @NicoDucou @AngelFQC