Updated all remaining TODOs.

Author: sebinside

src/main/scala/org/codeoverflow/chatoverflow/connector/ConnectorRegistry.scala

@@ -41,7 +41,6 @@ object ConnectorRegistry extends WithLogger {
         logger warn "Unable to remove connector. Connector is running."
         false
       } else {
-        // TODO: Check if this might crash anything
         connectors -= connectorKey
         true
       }

src/main/scala/org/codeoverflow/chatoverflow/connector/actor/FileSystemActor.scala

@@ -15,7 +15,6 @@ import scala.io.Source
   */
 class FileSystemActor extends Actor {
 
-  // TODO: Should be tested
   private val dataFilePath = Launcher.pluginDataPath
 
   // Create data folder if non existent

src/main/scala/org/codeoverflow/chatoverflow/framework/helper/PluggableLoader.scala

@@ -25,7 +25,6 @@ class PluggableLoader(classLoader: ClassLoader) extends WithLogger {
     // Now: Try to instantiate the pluggable class
     val pluggableObjects = createPluggableObjects(pluggableClasses)
 
-    // TODO: Are more than 1 implementation of pluggable allowed / possible / useful
     pluggableObjects.length match {
       case 0 => logger warn "No pluggable classes found."
       case 1 =>

src/main/scala/org/codeoverflow/chatoverflow/registry/TypeRegistry.scala

@@ -53,7 +53,7 @@ class TypeRegistry(requirementPackage: String) extends WithLogger {
     connectorTypes.clear()
 
     // Use reflection magic to get all impl-annotated classes
-    // FIXME: Does also find definitions not in the exact package - no problem right now
+    // FIXME: Does also find definitions not in the exact package - not intended should be filtered afterwards
     val reflections: Reflections = new Reflections(new ConfigurationBuilder()
       .setUrls(ClasspathHelper.forPackage(requirementPackage))
       .setScanners(new SubTypesScanner(), new TypeAnnotationsScanner()))

src/main/scala/org/codeoverflow/chatoverflow/requirement/service/file/FileConnector.scala

@@ -28,8 +28,6 @@ class FileConnector(override val sourceIdentifier: String) extends Connector(sou
     }
   }
 
-  // TODO: Save / Load XML
-
   def saveFile(pathInResources: String, content: String): Boolean = fileActor.??[Boolean](5){SaveFile(pathInResources, content)}.get
 
   def saveBinaryFile(pathInResources: String, content: Array[Byte]): Boolean = fileActor.??[Boolean](5){SaveBinaryFile(pathInResources, content)}.get