Skip to content
This repository has been archived by the owner on Aug 12, 2024. It is now read-only.
Nick Freear edited this page Aug 29, 2015 · 1 revision

I've extracted all the TODO comments from the code-base, and I'm putting them here, prior to deleting from the code (for code-quality reasons).

Typical grep reporting command:

grep -nri -A 1 TODO application

Todos - application

Note, lots of todos in svgweb have been removed to reduce distraction.

application/assets/ouplayer/ouice-dark/ouice-dark.css:67:/*TODO: add :active and :focus !!*/
application/assets/ouplayer/ouice-dark/ouice-dark.css-68-.oup-controls .play, .oupc .play:hover{ background-position:-2px -1px}
--
application/assets/ouplayer/ouice-light/ouice-light.css:35:/* TODO - ok? */
application/assets/ouplayer/ouice-light/ouice-light.css-36-#oup-tooltips, #fixedtipdiv{ background:#183440; color:#eee; text-shadow:none; }
--
application/assets/ouplayer/ouplayer.behaviors.js:109:    //TODO: Explicitly add/remove wait-cursor/ spinner.
application/assets/ouplayer/ouplayer.behaviors.js-110-    setTimeout(function(){ply.style.cursor='default';}, 4000);
--
application/assets/ouplayer/ouplayer.behaviors.js:311:	//TODO: check minimum Flash requirement!
application/assets/ouplayer/ouplayer.behaviors.js-312-	return (flashembed.isSupported([6,0,65]) && ua.indexOf('Android')===-1/* && ua.indexOf('WebKit')!==-1*/);
--
application/assets/ouplayer/ouplayer.tooltips.js:88://TODO: voffset
application/assets/ouplayer/ouplayer.tooltips.js-89-OUP.fixedtooltip=function(menucontents, obj, e, voffset, tipwidth, className){
--
application/assets/svgweb/COPYING.txt:59:TODO: There are alot of third party SVG files in samples/svg-files. Document where these came from, who the copyright holder is, and ensure we have a CLA on file for them.
application/assets/svgweb/COPYING.txt-60-
--
application/assets/svgweb/COPYING.txt:61:TODO: Figure out copyright information on the *.flv and *.mp3 files that we use in our samples in samples/svg-files
application/assets/svgweb/COPYING.txt-62-
--
application/assets/svgweb/COPYING.txt:63:TODO: Review the Flash code and identify any code that was either taken from other Flash libraries, blog posts, or tutorials. Document this and ensure we have appropriate CLAs from third-party code contributors.
application/assets/svgweb/COPYING.txt-64-
--
application/assets/svgweb/COPYING.txt:65:TODO: If you borrowed any code while creating your Flash ActionScript document it here, where it came from, version, etc. This is good open source hygiene :)
--
application/assets/svgweb/src/svg-uncompressed.htc:33:    <!-- TODO: Implement -->
application/assets/svgweb/src/svg-uncompressed.htc-34-    <!--
--
application/assets/svgweb/src/svg-uncompressed.htc:152:  // TODO: Implement
application/assets/svgweb/src/svg-uncompressed.htc-153-  /*
--
application/assets/svgweb/src/svg-uncompressed.js:1961:        // FIXME: TODO: can we eliminate having to generate these for all nodes
application/assets/svgweb/src/svg-uncompressed.js-1962-        // by changing the Flash backend to use GUIDs for these scenarios?

  ...

--
application/bin/xgettext.php:58:  /*@todo --keyword=plural' doesn't work :( --keyword=t:1, plural:1,2  --flag=plural:1:pass-c-format */
application/bin/xgettext.php-59-    $command = <<<EOF
--
application/config/routes.php:60:#$route['scripts/jquery.oembed.min.js'] = 'scripts/jquery_oembed'; #_jsmin'; #Todo.
application/config/routes.php-61-
--
application/controllers/admin.php:33:    * @todo - Improve content + styling, cf. http://cloudworks.ac.uk/admin/phpinfo
application/controllers/admin.php-34-    */
--
application/controllers/demo.php:64:      //Todo: ou-specific REGEX in config?
application/controllers/demo.php-65-        if ($player_url && !preg_match('@^http:\/\/mediaplayer\.open\.(ac.uk|edu)$@', $player_url)) {
--
application/controllers/embed.php:238:        //TODO: needs more work!
application/controllers/embed.php-239-        'popup_url' => site_url("popup/vle?media_url=").urlencode($player->media_url).'&title='.urlencode($player->title).'&'.$this->options_build_query(),
--
application/controllers/oembed.php:32:        //TODO - UA-, URL. ($this->uri->site_url() ?)
application/controllers/oembed.php-33-        if (isset($provider->_google_analytics)) {
--
application/core/MY_Lang.php:34:  * TODO: add 'Accept-Language' based negotiation - cf. CloudEngine.
application/core/MY_Lang.php:35:  * TODO: sanitize 'lang', tidy up paths, error handling.
application/core/MY_Lang.php-36-  */
--
application/views/ouplayer/oup_bodyclass.php:1:<?php
  //TODO: move $body_classes to controller function?

  // Add switches to body-class (no 'hulu').
  $body_classes = "oup mtype-$meta->media_type width-$meta->width theme-{$theme->name} hide-tscript hide-captions hide-settings oup-paused ";

  ...

--
application/views/ouplayer/oup_restricted.php:31:$about_url= isset($docs['about'])? $docs['about']: '#about/TODO';
application/views/ouplayer/oup_restricted.php-32-
--
application/views/ouplayer/oup_settings.php:64:$help_url = isset($docs['help']) ? $docs['help'] : '#help/TODO';
application/views/ouplayer/oup_settings.php:65:$about_url= isset($docs['about'])? $docs['about']: '#about/TODO';
application/views/ouplayer/oup_settings.php:66:$embedopts_url = isset($docs['embed']) ? $docs['embed'] : '#embed/TODO';
application/views/ouplayer/oup_settings.php-67-
--
application/views/site_layout/layout_ouvle.php:196:M.str = {"repository":{"add":"Add","back":"&laquo; Back","close":"Close","cleancache":"Clean my cache files","copying":"Copying","date":"Date","downloadsucc":"The file has been downloaded successfully","emptylist":"Empty list","error":"An unknown error occurred!","federatedsearch":"Federated search","filenotnull":"You must select a file to upload.","getfile":"Select this file","iconview":"View as icons","invalidjson":"Invalid JSON string","linkexternal":"Link external","listview":"View as list","loading":"Loading...","login":"Login","logout":"Logout","noenter":"Nothing entered","noresult":"No search result","manageurl":"Manage","popup":"Click \"Login\" button to login","preview":"Preview","refresh":"Refresh","save":"Save","saveas":"Save as","saved":"Saved","saving":"Saving","search":"Search","searching":"Search in","size":"Size","submit":"Submit","sync":"Sync","title":"Choose a file...","upload":"Upload this file","uploading":"Uploading...","xhtmlerror":"You are probably using XHTML strict header, some YUI Component doesn't work in this mode, please turn it off in moodle","chooselicense":"Choose license","author":"Author","norepositoriesavailable":"Sorry, none of your current repositories can return files in the required format.","norepositoriesexternalavailable":"Sorry, none of your current repositories can return external files.","nofilesattached":"No files attached","filepicker":"File picker","nofilesavailable":"No files available","overwrite":"Overwrite","renameto":"Rename to","fileexists":"File name already being used, please use another name","fileexistsdialogheader":"File exists","fileexistsdialog_editor":"A file with that name has already been attached to the text you are editing.","fileexistsdialog_filemanager":"A file with that name has already been attached"},"moodle":{"cancel":"Cancel","help":"Help","ok":"OK","error":"Error","info":"Information","viewallcourses":"View all courses","yes":"Yes"},"block":{"addtodock":"Move this to the dock","undockitem":"Undock this item","undockall":"Undock all"},"langconfig":{"thisdirectionvertical":"btt"},"admin":{"confirmation":"Confirmation"}};
application/views/site_layout/layout_ouvle.php-197-//]]>
--
application/views/test/player-error-test.php:65:Todo:
application/views/test/player-error-test.php-66-
--
Clone this wiki locally