MNT Remove TODO comments

Author: Sabina Talipova

client/src/components/UploadField/UploadField.js

@@ -226,7 +226,6 @@ class UploadField extends Component {
       return;
     }
 
-    // @todo - Should successful uploads be moved to another state?
     this.props.actions.uploadField.succeedUpload(this.props.id, file._queuedId, json[0]);
   }
 

client/src/components/UploadField/UploadFieldItem.scss

@@ -98,7 +98,6 @@ $uploadfield-item-label-height: 40px;
 
 // ACTIONS
 // Hidden checkbox is controlled via it's label
-// @todo - need this?
 .uploadfield-item__remove-btn {
   margin: 0;
 }

client/src/containers/AssetAdmin/AssetAdmin.js

@@ -283,8 +283,6 @@ class AssetAdmin extends Component {
           this.handleOpenFile(response.record.id);
         }
 
-        // TODO Update GraphQL store with new model,
-        // see https://github.com/silverstripe/silverstripe-graphql/issues/14
         return this.props.actions.files.readFiles()
           .then(() => {
             // open the containing folder, since folder edit mode isn't desired
@@ -477,9 +475,6 @@ class AssetAdmin extends Component {
    */
   handleUnpublish(fileIds) {
     return this.doUnpublish(fileIds).then((response) => {
-      // TODO Update GraphQL store with new model or update apollo and use new API
-      // see https://github.com/silverstripe/silverstripe-graphql/issues/14
-      // see https://dev-blog.apollodata.com/apollo-clients-new-imperative-store-api-6cb69318a1e3
       const { fileId } = this.props;
       this.props.actions.files.readFiles()
         .then(() => {
@@ -536,8 +531,7 @@ class AssetAdmin extends Component {
   }
 
   handleUpload() {
-    // TODO Update GraphQL store with new model,
-    // see https://github.com/silverstripe/silverstripe-graphql/issues/14
+    // noop
   }
 
   handleUploadQueue() {
@@ -558,7 +552,6 @@ class AssetAdmin extends Component {
   }
 
   handleMoveFilesSuccess(folderId, fileIds) {
-    // TODO Refactor "queued files" into separate visual area and remove coupling here
     const files = this.props.queuedFiles.items.filter((file) => (
       fileIds.includes(file.id)
     ));
@@ -570,8 +563,6 @@ class AssetAdmin extends Component {
     });
 
     this.props.actions.gallery.deselectFiles();
-    // TODO Update GraphQL store with new model,
-    // see https://github.com/silverstripe/silverstripe-graphql/issues/14
     this.props.actions.files.readFiles();
   }
 
@@ -799,7 +790,6 @@ function mapStateToProps(state, ownProps) {
   const { formSchema } = state.assetAdmin.modal;
   return {
     securityId: state.config.SecurityID,
-    // TODO Refactor "queued files" into separate visual area and remove coupling here
     queuedFiles: state.assetAdmin.queuedFiles,
     showSearch: state.assetAdmin.displaySearch.isOpen,
     type: (formSchema && formSchema.type) || ownProps.type,
@@ -812,7 +802,6 @@ function mapDispatchToProps(dispatch) {
       gallery: bindActionCreators(galleryActions, dispatch),
       toasts: bindActionCreators(toastsActions, dispatch),
       displaySearch: bindActionCreators(displaySearchActions, dispatch),
-      // TODO Refactor "queued files" into separate visual area and remove coupling here
       queuedFiles: bindActionCreators(queuedFilesActions, dispatch),
       confirmDeletion: bindActionCreators(confirmDeletionActions, dispatch)
     },

client/src/containers/Editor/LegacyPopoverField.scss

@@ -40,7 +40,6 @@
 
     // Unable to use classes within the popover, so we use elements to style
     ul {
-      // TODO remove important by removing nesting of lists
       padding-left: 0 !important;
       list-style-type: none;
       margin-left: -#{$popover-body-padding-x} + 1px; // minus border

client/src/containers/Gallery/Gallery.js

@@ -127,27 +127,23 @@ class Gallery extends Component {
     }
 
     if (filters.lastEditedFrom && filters.lastEditedTo) {
-      // TODO Date localisation
       messages.push(i18n._t(
         'AssetAdmin.SEARCHRESULTSMESSAGEEDITEDBETWEEN',
         'last edited between \'{lastEditedFrom}\' and \'{lastEditedTo}\''
       ));
     } else if (filters.lastEditedFrom) {
-      // TODO Date localisation
       messages.push(i18n._t(
         'AssetAdmin.SEARCHRESULTSMESSAGEEDITEDFROM',
         'last edited after \'{lastEditedFrom}\''
       ));
     } else if (filters.lastEditedTo) {
-      // TODO Date localisation
       messages.push(i18n._t(
         'AssetAdmin.SEARCHRESULTSMESSAGEEDITEDTO',
         'last edited before \'{lastEditedTo}\''
       ));
     }
 
     if (filters.appCategory) {
-      // TODO Category name localisation
       messages.push(i18n._t(
         'AssetAdmin.SEARCHRESULTSMESSAGECATEGORY',
         'categorised as \'{appCategory}\''
@@ -402,8 +398,6 @@ class Gallery extends Component {
 
     this.props.actions.queuedFiles.succeedUpload(fileXhr._queuedId, json[0]);
 
-    // TODO Update GraphQL store with new model,
-    // see https://github.com/silverstripe/silverstripe-graphql/issues/14
     if (this.props.onSuccessfulUpload) {
       this.props.onSuccessfulUpload(json);
     }

client/src/containers/HistoryList/HistoryItem.js

@@ -25,7 +25,6 @@ class HistoryItem extends Component {
     }
 
     return (
-      // @todo wrap the contents in an `<a>` tag
       // eslint-disable-next-line jsx-a11y/no-noninteractive-element-interactions
       <li
         className="list-group-item history-item"

client/src/containers/HistoryList/HistoryList.js

@@ -11,8 +11,6 @@ const sectionConfigKey = 'SilverStripe\\AssetAdmin\\Controller\\AssetAdmin';
 /**
  * Create a new endpoint
  *
- * @todo duplication with assetadmin.
- *
  * @param {Object} endpointConfig
  * @param {Boolean} includeToken
  * @returns {Function}
@@ -47,7 +45,7 @@ class HistoryList extends Component {
   }
 
   componentDidUpdate(prevProps) {
-    // TODO race conditions happening, this should have history state shifted to redux
+    // Avoids race conditions from happening
     this.refreshHistoryIfNeeded(prevProps);
   }
 
@@ -75,7 +73,6 @@ class HistoryList extends Component {
        * This needs a delay/throttle, so this api request tries to be made last in the stack.
        * We also use this to stop an API call happening if the component is going to
        * unmount soon.
-       * TODO: This could potentially be solved by using apollo-client's caching and graphql.
        */
       this.timer = setTimeout(() => {
         this.api({

client/src/containers/TableView/TableView.js

@@ -131,7 +131,6 @@ class TableView extends Component {
       columnMetadata: this.getColumnConfig(),
       useGriddleStyles: false,
       onRowClick: this.handleRowClick,
-      // TODO will need to request upstream to stop their internal sorting to show folders first
       results: this.props.files,
       customNoDataComponent: this.renderNoItemsNotice,
     };

client/src/entwine/TinyMCE_ssmedia.js

@@ -271,10 +271,6 @@ jQuery.entwine('ss', ($) => {
     _handleInsert(data, file) {
       let result = false;
       this.setData(Object.assign({}, data, file));
-
-      // Sometimes AssetAdmin.js handleSubmitEditor() can't find the file
-      // @todo Ensure that we always return a file for any valid ID
-
       // in case of any errors, better to catch them than let them go silent
       try {
         let category = null;

client/src/entwine/UploadField/UploadFieldEntwine.js

@@ -68,7 +68,6 @@ jQuery.entwine('ss', ($) => {
 
       const UploadField = this.getComponent();
 
-      // TODO: rework entwine so that react has control of holder
       let root = this.getReactRoot();
       if (!root) {
         root = createRoot(this.getContainer());