Miqra-Engineering/moodle_local_treestudyplan
2
0
Fork 0
Code Issues 13 Pull Requests Packages Projects Releases 4 Wiki Activity

POST params processing #78

New Issue
Closed
opened 2024-10-28 20:58:22 +01:00 by vldovhan · 1 comment
vldovhan commented 2024-10-28 20:58:22 +01:00
Copy Link

Hello, you are using PHP native post parameters cleaning in your plugin:
image

As an improvement, please consider using Moodle functions, like clean_param() and clean_param_array().

Hello, you are using PHP native post parameters cleaning in your plugin: ![image](/attachments/d999e4d3-bae2-4c01-9b4d-ea995820d694) As an improvement, please consider using Moodle functions, like clean_param() and clean_param_array().
image.png
110 KiB
pmkuipers self-assigned this 2024-11-01 13:10:19 +01:00
pmkuipers added the
Fixed
Confirmed
 label 2024-11-01 18:04:18 +01:00
pmkuipers commented 2024-11-01 18:04:38 +01:00
Owner
Copy Link

Resolved in commit 5e3d610451 by switching to moodle param requests

Resolved in commit 5e3d610451 by switching to moodle param requests
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
Fixed
Confirmed
The issue has been fixed and the fix has been tested

  
Fixed
Unconfirmed
The issue has been fixed, but the fix needs more testing

  
Kind
Breaking
Breaking change that won't be backward compatible

  
Kind
Bug
Something is not working

  
Kind
Deprecation issue
Address use of deprecated features in libraries

  
Kind
Documentation
Documentation changes

  
Kind
Enhancement
Improve existing functionality

  
Kind
Feature
New functionality

  
Kind
Language
Issue is related to a typo or a need to reword text

  
Kind
Security
This is security issue

  
Priority
High

  
Priority
Low

  
Reviewed
Confirmed
Issue has been confirmed

  
Reviewed
Duplicate
This issue or pull request already exists

  
Reviewed
Invalid
Invalid issue

  
Reviewed
Won't Fix
This issue won't be fixed

  
Status
Abandoned
Somebody has started to work on this but abandoned work

  
Status
Blocked
Something is blocking this issue or pull request

  
Status
Need More Info
Feedback is required to reproduce issue or to continue work

No Label
Fixed
Confirmed
Fixed
Unconfirmed
Kind
Breaking
Kind
Bug
Kind
Deprecation issue
Kind
Documentation
Kind
Enhancement
Kind
Feature
Kind
Language
Kind
Security
Priority
High
Priority
Low
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
pmkuipers
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Miqra-Engineering/moodle_local_treestudyplan#78
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?