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

CTP-3690 automated code checker fixes #12

Merged
merged 25 commits into from
Sep 10, 2024
Merged

Conversation

watson8
Copy link
Contributor

@watson8 watson8 commented Sep 2, 2024

No description provided.

@watson8 watson8 requested a review from aspark21 September 2, 2024 14:56
@watson8 watson8 force-pushed the CTP-3690-code-checker-2 branch from 055afb8 to b84653b Compare September 3, 2024 10:07
@watson8 watson8 changed the title CTP-3690 multi line array comma CTP-3690 automated code checker fixes Sep 3, 2024
actions/allocate.php Outdated Show resolved Hide resolved
$csv_cell = substr($csv_cell, 0, -1);
foreach ($csvcells as $csvcell) {
if (substr($csvcell, 0, 8) == 'assessor') {
$stagedentifier = 'assessor_'.(substr($csvcell, -1));
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dentifier?

@@ -250,7 +250,7 @@ public function get_stage_identifier_for_assessor($submission, $student) {
* Function to validate cell for the file upload
* @return mixed
*/
public function validate_cell($value, $submissions, $stage_dentifier='', $uploadedgradecells = []) {
public function validate_cell($value, $submissions, $stagedentifier='', $uploadedgradecells = []) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dentifier again

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These look like typos in the original code. I need to do another PR to clear up the files I left to avoid conflict to will fix those in there

@@ -47,6 +47,6 @@ public function get_header($stage);
* @param $stage_dentifier
* @return mixed
*/
public function validate_cell($value, $submissions, $stage_dentifier='');
public function validate_cell($value, $submissions, $stagedentifier='');
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh identifier AND dentifier

@@ -1220,34 +1224,34 @@ protected function render_resubmit_to_plagiarism_button($coursework, $submission
protected function coursework_deadlines_table(mod_coursework_coursework $coursework) {
global $USER;

$dealine_extension =
$dealineextension =
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dealine

@aspark21 aspark21 merged commit 45eb6f3 into main Sep 10, 2024
0 of 6 checks passed
@watson8 watson8 deleted the CTP-3690-code-checker-2 branch September 16, 2024 15:24
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

Successfully merging this pull request may close these issues.

2 participants