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

fix: Fixes module URLs and activity types to close #186. #221

Merged
merged 32 commits into from
Aug 3, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
4a166dd
fix: Fixes activity definition type for facetoface transformers.
ryasmi Aug 3, 2018
1a78327
fix: Fixes course_module_viewed activity definition type for book.
ryasmi Aug 3, 2018
12a6401
fix: Fixes course_module_viewed activity definition type for chat.
ryasmi Aug 3, 2018
538feeb
fix: Fixes course_module_viewed activity definition type for choice.
ryasmi Aug 3, 2018
0341575
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
1fc9193
fix: Fixes course_module_viewed activity definition type for facetoface.
ryasmi Aug 3, 2018
ad9f491
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
8f7fdf9
refactor: Changes the course_feedback util to use the course_module u…
ryasmi Aug 3, 2018
c79202a
refactor: Changes the course_feedback util to use the course_module u…
ryasmi Aug 3, 2018
7d698ed
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
445d64b
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
d6e0350
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
c1ac049
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
df4ce5c
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
e15af12
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
e9734c9
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
94e0eaa
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
173dd38
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
9966ab3
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
9af35e8
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
55587cd
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
d00618f
test: Adds test for course_module_viewed activity definition type for…
ryasmi Aug 3, 2018
e1b99e7
fix: Fixes course_module_viewed activity definition type for lesson.
ryasmi Aug 3, 2018
2a98316
fix: Fixes course_module_viewed activity definition type for page.
ryasmi Aug 3, 2018
30814df
fix: Fixes course_module_viewed activity definition type for quiz.
ryasmi Aug 3, 2018
f1c70e8
fix: Fixes course_module_viewed activity definition type for resource.
ryasmi Aug 3, 2018
45a7ab4
fix: Fixes course_module_viewed activity definition type for scorm.
ryasmi Aug 3, 2018
852fc52
fix: Fixes course_module_viewed activity definition type for survey.
ryasmi Aug 3, 2018
9314a0c
fix: Fixes course_module_viewed activity definition type for url.
ryasmi Aug 3, 2018
65eec8c
fix: Fixes attempt_viewed activity definition type for quiz.
ryasmi Aug 3, 2018
b9bf02f
refactor: Removes unused module util.
ryasmi Aug 3, 2018
73f8370
ci(lint): Fixes linting errors.
ryasmi Aug 3, 2018
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 6 additions & 1 deletion src/transformer/events/all/course_module_viewed.php
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,12 @@ function course_module_viewed(array $config, \stdClass $event) {
$lang => 'viewed'
],
],
'object' => utils\get_activity\event_module($config, $event, $lang),
'object' => utils\get_activity\course_module(
$config,
$course,
$event->contextinstanceid,
'http://id.tincanapi.com/activitytype/lms/module'
),
'timestamp' => utils\get_event_timestamp($event),
'context' => [
'platform' => $config['source_name'],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,12 @@ function course_module_completion_updated(array $config, \stdClass $event) {
$lang => 'completed'
],
],
'object' => utils\get_activity\module($config, $moduletype->name, $module, $lang),
'object' => utils\get_activity\course_module(
$config,
$course,
$event->contextinstanceid,
'http://id.tincanapi.com/activitytype/lms/module'
),
'timestamp' => utils\get_event_timestamp($event),
'context' => [
'platform' => $config['source_name'],
Expand Down
61 changes: 61 additions & 0 deletions src/transformer/events/mod_book/course_module_viewed.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
<?php
// This file is part of Moodle - http://moodle.org/
//
// Moodle is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// Moodle is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

namespace src\transformer\events\mod_book;

defined('MOODLE_INTERNAL') || die();

use src\transformer\utils as utils;

function course_module_viewed(array $config, \stdClass $event) {
$repo = $config['repo'];
$user = $repo->read_record_by_id('user', $event->userid);
$course = $repo->read_record_by_id('course', $event->courseid);
$lang = utils\get_course_lang($course);

return [[
'actor' => utils\get_user($config, $user),
'verb' => [
'id' => 'http://id.tincanapi.com/verb/viewed',
'display' => [
$lang => 'viewed'
],
],
'object' => utils\get_activity\course_module(
$config,
$course,
$event->contextinstanceid,
'http://id.tincanapi.com/activitytype/book'
),
'timestamp' => utils\get_event_timestamp($event),
'context' => [
'platform' => $config['source_name'],
'language' => $lang,
'extensions' => [
utils\INFO_EXTENSION => utils\get_info($config, $event),
],
'contextActivities' => [
'grouping' => [
utils\get_activity\site($config),
utils\get_activity\course($config, $course),
],
'category' => [
utils\get_activity\source($config),
]
],
]
]];
}
61 changes: 61 additions & 0 deletions src/transformer/events/mod_chat/course_module_viewed.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
<?php
// This file is part of Moodle - http://moodle.org/
//
// Moodle is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// Moodle is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

namespace src\transformer\events\mod_chat;

defined('MOODLE_INTERNAL') || die();

use src\transformer\utils as utils;

function course_module_viewed(array $config, \stdClass $event) {
$repo = $config['repo'];
$user = $repo->read_record_by_id('user', $event->userid);
$course = $repo->read_record_by_id('course', $event->courseid);
$lang = utils\get_course_lang($course);

return [[
'actor' => utils\get_user($config, $user),
'verb' => [
'id' => 'http://id.tincanapi.com/verb/viewed',
'display' => [
$lang => 'viewed'
],
],
'object' => utils\get_activity\course_module(
$config,
$course,
$event->contextinstanceid,
'http://id.tincanapi.com/activitytype/chat-channel'
),
'timestamp' => utils\get_event_timestamp($event),
'context' => [
'platform' => $config['source_name'],
'language' => $lang,
'extensions' => [
utils\INFO_EXTENSION => utils\get_info($config, $event),
],
'contextActivities' => [
'grouping' => [
utils\get_activity\site($config),
utils\get_activity\course($config, $course),
],
'category' => [
utils\get_activity\source($config),
]
],
]
]];
}
7 changes: 6 additions & 1 deletion src/transformer/events/mod_facetoface/cancel_booking.php
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,12 @@ function cancel_booking(array $config, \stdClass $event) {
$lang => 'unregistered from'
],
],
'object' => utils\get_activity\event_module($config, $event, $lang),
'object' => utils\get_activity\course_module(
$config,
$course,
$event->contextinstanceid,
'https://w3id.org/xapi/acrossx/activities/face-to-face-discussion'
),
'timestamp' => utils\get_event_timestamp($event),
'context' => [
'platform' => $config['source_name'],
Expand Down
61 changes: 61 additions & 0 deletions src/transformer/events/mod_facetoface/course_module_viewed.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,61 @@
<?php
// This file is part of Moodle - http://moodle.org/
//
// Moodle is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// Moodle is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

namespace src\transformer\events\mod_facetoface;

defined('MOODLE_INTERNAL') || die();

use src\transformer\utils as utils;

function course_module_viewed(array $config, \stdClass $event) {
$repo = $config['repo'];
$user = $repo->read_record_by_id('user', $event->userid);
$course = $repo->read_record_by_id('course', $event->courseid);
$lang = utils\get_course_lang($course);

return [[
'actor' => utils\get_user($config, $user),
'verb' => [
'id' => 'http://id.tincanapi.com/verb/viewed',
'display' => [
$lang => 'viewed'
],
],
'object' => utils\get_activity\course_module(
$config,
$course,
$event->contextinstanceid,
'https://w3id.org/xapi/acrossx/activities/face-to-face-discussion'
),
'timestamp' => utils\get_event_timestamp($event),
'context' => [
'platform' => $config['source_name'],
'language' => $lang,
'extensions' => [
utils\INFO_EXTENSION => utils\get_info($config, $event),
],
'contextActivities' => [
'grouping' => [
utils\get_activity\site($config),
utils\get_activity\course($config, $course),
],
'category' => [
utils\get_activity\source($config),
]
],
]
]];
}
7 changes: 6 additions & 1 deletion src/transformer/events/mod_facetoface/signup_success.php
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,12 @@ function signup_success(array $config, \stdClass $event) {
$lang => 'registered to'
],
],
'object' => utils\get_activity\event_module($config, $event, $lang),
'object' => utils\get_activity\course_module(
$config,
$course,
$event->contextinstanceid,
'https://w3id.org/xapi/acrossx/activities/face-to-face-discussion'
),
'timestamp' => utils\get_event_timestamp($event),
'context' => [
'platform' => $config['source_name'],
Expand Down
7 changes: 6 additions & 1 deletion src/transformer/events/mod_facetoface/take_attendance.php
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,12 @@ function take_attendance(array $config, \stdClass $event) {
$lang => 'attended'
],
],
'object' => utils\get_activity\event_module($config, $event, $lang),
'object' => utils\get_activity\course_module(
$config,
$course,
$event->contextinstanceid,
'https://w3id.org/xapi/acrossx/activities/face-to-face-discussion'
),
'timestamp' => utils\get_event_timestamp($event),
'result' => [
'duration' => "PT".(string) $sessionduration."S",
Expand Down
3 changes: 1 addition & 2 deletions src/transformer/events/mod_feedback/course_module_viewed.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ function course_module_viewed(array $config, \stdClass $event) {
$repo = $config['repo'];
$user = $repo->read_record_by_id('user', $event->userid);
$course = $repo->read_record_by_id('course', $event->courseid);
$feedback = $repo->read_record_by_id('feedback', $event->objectid);
$lang = utils\get_course_lang($course);

return [[
Expand All @@ -35,7 +34,7 @@ function course_module_viewed(array $config, \stdClass $event) {
$lang => 'viewed'
],
],
'object' => utils\get_activity\course_feedback($config, $event->contextinstanceid, $feedback, $lang),
'object' => utils\get_activity\course_feedback($config, $course, $event->contextinstanceid),
'timestamp' => utils\get_event_timestamp($event),
'context' => [
'platform' => $config['source_name'],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ function handler(array $config, \stdClass $event, \stdClass $feedbackvalue) {
case 'multichoice':
return multichoice($config, $event, $feedbackvalue, $feedbackitem);
case 'numeric':
return numeric($config, $event, $feedbackvalue, $feedbackitem);
return numerical($config, $event, $feedbackvalue, $feedbackitem);
case 'textarea':
return textarea($config, $event, $feedbackvalue, $feedbackitem);
case 'textfield':
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ function multichoice(array $config, \stdClass $event, \stdClass $feedbackvalue,
'grouping' => [
utils\get_activity\site($config),
utils\get_activity\course($config, $course),
utils\get_activity\course_feedback($config, $event->contextinstanceid, $feedback, $lang),
utils\get_activity\course_feedback($config, $course, $event->contextinstanceid),
],
'category' => [
utils\get_activity\source($config),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ function multichoicerated(array $config, \stdClass $event, \stdClass $feedbackva
'grouping' => [
utils\get_activity\site($config),
utils\get_activity\course($config, $course),
utils\get_activity\course_feedback($config, $event->contextinstanceid, $feedback, $lang),
utils\get_activity\course_feedback($config, $course, $event->contextinstanceid),
],
'category' => [
utils\get_activity\source($config),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@

use src\transformer\utils as utils;

function numeric(array $config, \stdClass $event, \stdClass $feedbackvalue, \stdClass $feedbackitem) {
function numerical(array $config, \stdClass $event, \stdClass $feedbackvalue, \stdClass $feedbackitem) {
$repo = $config['repo'];
$user = $repo->read_record_by_id('user', $event->userid);
$course = $repo->read_record_by_id('course', $event->courseid);
Expand Down Expand Up @@ -63,7 +63,7 @@ function numeric(array $config, \stdClass $event, \stdClass $feedbackvalue, \std
'grouping' => [
utils\get_activity\site($config),
utils\get_activity\course($config, $course),
utils\get_activity\course_feedback($config, $event->contextinstanceid, $feedback, $lang),
utils\get_activity\course_feedback($config, $course, $event->contextinstanceid),
],
'category' => [
utils\get_activity\source($config),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ function textarea(array $config, \stdClass $event, \stdClass $feedbackvalue, \st
'grouping' => [
utils\get_activity\site($config),
utils\get_activity\course($config, $course),
utils\get_activity\course_feedback($config, $event->contextinstanceid, $feedback, $lang),
utils\get_activity\course_feedback($config, $course, $event->contextinstanceid),
],
'category' => [
utils\get_activity\source($config),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ function textfield(array $config, \stdClass $event, \stdClass $feedbackvalue, \s
'grouping' => [
utils\get_activity\site($config),
utils\get_activity\course($config, $course),
utils\get_activity\course_feedback($config, $event->contextinstanceid, $feedback, $lang),
utils\get_activity\course_feedback($config, $course, $event->contextinstanceid),
],
'category' => [
utils\get_activity\source($config),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ function response_submitted(array $config, \stdClass $event) {
$lang => 'submitted'
],
],
'object' => utils\get_activity\course_feedback($config, $event->contextinstanceid, $feedback, $lang),
'object' => utils\get_activity\course_feedback($config, $course, $event->contextinstanceid),
'timestamp' => utils\get_event_timestamp($event),
'context' => [
'platform' => $config['source_name'],
Expand Down
Loading