Skip to content

Commit

Permalink
MDL-44482 mod_lti: use localized dates
Browse files Browse the repository at this point in the history
Fixed an issue where LTI external tools had creation datestamps of NaN/NaN/NaN under particular non-English languages.
  • Loading branch information
Jetha Chan committed May 26, 2014
1 parent 529d4cd commit addeb65
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion mod/lti/locallib.php
Expand Up @@ -360,7 +360,7 @@ function lti_get_tool_table($tools, $id) {
";

foreach ($tools as $type) {
$date = userdate($type->timecreated);
$date = userdate($type->timecreated, get_string('strftimedatefullshort', 'core_langconfig'));
$accept = get_string('accept', 'lti');
$update = get_string('update', 'lti');
$delete = get_string('delete', 'lti');
Expand Down
4 changes: 2 additions & 2 deletions mod/lti/settings.php
Expand Up @@ -144,7 +144,7 @@
var configuredColumns = [
{key:'name', label:'$typename', sortable:true},
{key:'baseURL', label:'$baseurl', sortable:true},
{key:'timecreated', label:'$createdon', sortable:true, formatter:Y.YUI2.widget.DataTable.formatDate},
{key:'timecreated', label:'$createdon', sortable:true},
{key:'action', label:'$action'}
];
Expand All @@ -153,7 +153,7 @@
fields: [
{key:'name'},
{key:'baseURL'},
{key:'timecreated', parser:'date'},
{key:'timecreated'},
{key:'action'}
]
};
Expand Down

0 comments on commit addeb65

Please sign in to comment.