The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.
| Tag Class | Total number of occurrences | Tag strings used by tag class | 
|---|---|---|
| Todo Work | 160 | todo, FIXME, XXX | 
Each tag is detailed below:
Number of occurrences found in the code: 160
| ch.ethz.iks.slp.impl.SLPConfiguration | Line | 
|---|---|
| wire this to the properties if necessary | 530 | 
| ch.ethz.iks.slp.impl.SLPDaemonImpl | Line | 
| the RFC encourages to keep the connection open to allow the other side to send multiple requests per connection. So start a server thread for every incoming connection instead of closing the connection after the first request | 512 | 
| --No comment-- | 530 | 
| ch.ethz.iks.slp.impl.Service | Line | 
| support localized registrations ... | 60 | 
| org.universAAL.middleware.api.SimpleServiceRegistrator | Line | 
| convert lists | 412 | 
| org.universAAL.middleware.brokers.control.ControlBroker | Line | 
| check this, the deploy connector is currently only available in Karaf || getDeployConnector() == null | 175 | 
| handle ConfigurationManagerConnector? | 271 | 
| Handle local uninstallation | 522 | 
| we should send a part in small pieces or by means of out-of-band protocol which support streaming | 593 | 
| Ask michele if it was missing by reason | 808 | 
| org.universAAL.middleware.brokers.control.WaitForResponse | Line | 
| Log me | 98 | 
| org.universAAL.middleware.brokers.message.configuration.ConfigurationMessage | Line | 
| Auto-generated catch block | 107 | 
| org.universAAL.middleware.brokers.message.control.ControlMessage | Line | 
| Auto-generated catch block | 114 | 
| Auto-generated method stub | 169 | 
| org.universAAL.middleware.brokers.message.deploy.DeployMessage | Line | 
| Auto-generated catch block | 77 | 
| Auto-generated method stub | 97 | 
| org.universAAL.middleware.brokers.message.space.SpaceMessage | Line | 
| Auto-generated catch block | 71 | 
| Auto-generated method stub | 87 | 
| org.universAAL.middleware.bus.model.AbstractBus | Line | 
| workaround for non-space-coordinators (space ID is unknown // then) SPACE_INSTANCE_URI_PREFIX = ""; | 141 | 
| should we handle duplicates? Assume a malicious component that creates its own bus and creates IDs in a loop, then the counter could overflow | 402 | 
| org.universAAL.middleware.bus.model.util.RegistryMap | Line | 
| synchronize map and listener for concurrent access | 39 | 
| org.universAAL.middleware.bus.permission.Permission | Line | 
| remove this when the broker-/channel-names are fixed | 76 | 
| org.universAAL.middleware.connectors.communication.jgroups.JGroupsCommunicationConnector | Line | 
| Add log message or error if some of the destination does not exists on the channels | 519 | 
| We should reset the configuration to the default properties | 581 | 
| Auto-generated method stub | 661 | 
| Auto-generated method stub | 665 | 
| Auto-generated method stub | 669 | 
| Auto-generated method stub | 676 | 
| Auto-generated method stub | 694 | 
| org.universAAL.middleware.connectors.discovery.slp.SLPBrowser | Line | 
| remove the static name of the channel name | 102 | 
| org.universAAL.middleware.connectors.util.ChannelMessage | Line | 
| Auto-generated catch block | 94 | 
| org.universAAL.middleware.connectors.util.Consts | Line | 
| to generalize | 39 | 
| org.universAAL.middleware.container.JUnit.JUnitModuleContext | Line | 
| Alternatively root logger could log to target/test.log | 82 | 
| org.universAAL.middleware.container.osgi.OSGiModuleContext | Line | 
| check permissions | 151 | 
| check permissions | 159 | 
| check permissions | 167 | 
| check permissions | 237 | 
| define a convention for the array param current assumption: 1st param @ index 0 is the org.osgi.framework.Constants.SERVICE_PID chosen for a org.osgi.service.cm.ManagedService (type = String) possible extensions: 2nd param @ index 1: help string describing the role of the property file indirectly specified by the first first param @ index 0 3rd param @ index 2: a hash-table with allowed properties as keys and a help string about each property as value | 289 | 
| log | 347 | 
| log | 362 | 
| log | 377 | 
| maybe set another system property to point to the main data folder | 445 | 
| org.universAAL.middleware.container.pojo.POJOContainer | Line | 
| this will maintain previous properties which are not overridden by the file. | 166 | 
| org.universAAL.middleware.container.pojo.layers.AbstractBusLayer | Line | 
| init card correctly? | 50 | 
| use real managers when set to do so. | 60 | 
| org.universAAL.middleware.container.test.ClassTest | Line | 
| Auto-generated method stub | 33 | 
| Auto-generated method stub | 38 | 
| org.universAAL.middleware.context.data.AbstractFiltererContainer | Line | 
| remove this class and move the containerKey to the FiltererContainer class | 30 | 
| org.universAAL.middleware.context.impl.ContextBusImpl | Line | 
| Auto-generated method stub | 163 | 
| org.universAAL.middleware.context.impl.ContextStrategy | Line | 
| --No comment-- | 439 | 
| --No comment-- | 443 | 
| org.universAAL.middleware.context.owl.ContextBusOntology | Line | 
| ContextEventPattern is not a ManagedIndividual! oci.addRestriction(password, Restriction.getAllValuesRestrictionWithCardinality(ContextProvider.PROP_CONTEXT_PROVIDED_EVENTS, ContextEventPattern.MY_URI, -1, 1)); | 79 | 
| org.universAAL.middleware.context.test.ArtifactIT | Line | 
| Test "OR" patterns (using several different CEPs) | 257 | 
| org.universAAL.middleware.interfaces.space.SpaceCard | Line | 
| Profile of the Space. Currently this is a string but a more constrained type should be used | 43 | 
| org.universAAL.middleware.managers.configuration.core.impl.RemoteConfigurationEntity | Line | 
| create filters | 86 | 
| add receivers. | 96 | 
| org.universAAL.middleware.managers.configuration.core.impl.secondaryManagers.EntityManager | Line | 
| , when they are not equal but the version is the same, there is a conflict! | 192 | 
| org.universAAL.middleware.managers.deploy.DeployManagerImpl | Line | 
| On failure of installation of a part we could retry up-to n-times | 321 | 
| On permanent failure we should roll back the installation by removing the parts from peers | 323 | 
| following aggregation has been done on purpose | 630 | 
| we should handle the above cases PART_NOT_UNINSTALLED, PART_UNINSTALLED | 635 | 
| Add check for the runtime support }catch | 899 | 
| Auto-generated method stub | 956 | 
| Auto-generated method stub | 963 | 
| Auto-generated method stub | 981 | 
| Auto-generated method stub | 985 | 
| Auto-generated method stub | 993 | 
| org.universAAL.middleware.managers.distributedmw.impl.BusMemberListenerHandler | Line | 
| performance - make this only once | 151 | 
| org.universAAL.middleware.managers.distributedmw.impl.DistributedMWManagerImpl | Line | 
| split this implementation in two for each interface for security reasons | 45 | 
| org.universAAL.middleware.managers.distributedmw.impl.ListenerHandler | Line | 
| subscribe for all allPeersListeners in all peers? or do we get a call to newPeerJoined? | 253 | 
| Auto-generated catch block | 282 | 
| org.universAAL.middleware.managers.distributedmw.impl.LogListenerHandler | Line | 
| log message? we received a message from a node to which we did not subscribe. This can also happen in the short time after removing the listener until the remote node stops sending messages | 74 | 
| org.universAAL.middleware.managers.space.util.SpaceSchemaEventHandler | Line | 
| Auto-generated method stub | 44 | 
| org.universAAL.middleware.managers.tenant.TenantManagerImpl | Line | 
| log me | 90 | 
| optimization instead of running through all the listeners and invoking newTenantRegistered, a thread can do this on the background | 100 | 
| log me | 106 | 
| org.universAAL.middleware.modules.communication.CommunicationModuleImpl | Line | 
| Auto-generated method stub | 301 | 
| Auto-generated method stub | 308 | 
| Auto-generated method stub | 315 | 
| org.universAAL.middleware.owl.BoundedValueRestriction | Line | 
| check other ClassExpressions (e.g. Union..) | 345 | 
| org.universAAL.middleware.owl.ComparableIndividual | Line | 
| change to getMaxValue? | 56 | 
| change to getMinValue? | 65 | 
| org.universAAL.middleware.owl.Complement | Line | 
| all values different from those in the changed hashtable would cause a match | 99 | 
| org.universAAL.middleware.owl.Enumeration | Line | 
| what if variables had to be replaced using context | 191 | 
| what if variables had to be replaced using context | 202 | 
| there is still a chance to return true... | 216 | 
| what if variables had to be replaced using context | 222 | 
| for complement and restriction difficult to decide but also very unlikely | 228 | 
| if cloned.size() != context.size(), then under certain conditions it could still work | 246 | 
| if cloned.size() != context.size(), then under certain conditions it could still work | 260 | 
| org.universAAL.middleware.owl.HasValueRestriction | Line | 
| because the equality was conditional, there is still a chance to return true by adopting complement conditions into context | 267 | 
| org.universAAL.middleware.owl.Intersection | Line | 
| if cloned.size() > context.size(), then under certain conditions it could still work | 203 | 
| there is still chance to return true | 207 | 
| org.universAAL.middleware.owl.LengthRestriction | Line | 
| check other ClassExpressions (e.g. Union..) | 515 | 
| org.universAAL.middleware.owl.ManagedIndividual | Line | 
| there is no owl:Individual, use owl:NamedIndividual? | 62 | 
| change this!! (not very performant, maybe remove this method?) | 223 | 
| only resources are copies, but not list of resources. | 347 | 
| will create an infinite loop for cycles. | 348 | 
| org.universAAL.middleware.owl.ObjectProperty | Line | 
| inverseOf; | 171 | 
| org.universAAL.middleware.owl.OntClassInfo | Line | 
| --No comment-- | 235 | 
| Auto-generated method stub | 240 | 
| Auto-generated method stub | 245 | 
| org.universAAL.middleware.owl.Ontology | Line | 
| lock/immutable info | 532 | 
| org.universAAL.middleware.owl.PropertyRestriction | Line | 
| change | 174 | 
| change | 199 | 
| should we really do this? | 249 | 
| should we merge? | 277 | 
| org.universAAL.middleware.owl.TypeRestriction | Line | 
| log message? | 138 | 
| log message? | 142 | 
| log message? | 227 | 
| check other ClassExpressions (e.g. Union..) | 568 | 
| org.universAAL.middleware.owl.TypeURI | Line | 
| 1. could variables be used in constructing class names? 2. what if variables are used not only as values but also within values | 174 | 
| there is still a chance to return true... so fall through to the general case at the end | 215 | 
| org.universAAL.middleware.owl.Union | Line | 
| there is still a chance to return true... now fall through to the general cases below to have more chance for correct answer | 162 | 
| the case, where the whole union is really the supertype of a complement, an intersection, a TypeURI, or a Restriction is still open. | 181 | 
| org.universAAL.middleware.owl.generic.GenericOntology | Line | 
| handle imports correctly -> should be done in Ontology class directly | 88 | 
| org.universAAL.middleware.owl.util.OntologyTest | Line | 
| there is much more we can test! | 344 | 
| org.universAAL.middleware.rdf.Property | Line | 
| Auto-generated method stub | 134 | 
| Auto-generated method stub | 142 | 
| org.universAAL.middleware.rdf.RDFClassInfo | Line | 
| will we have subclasses of Restriction? If yes, how to handle subclasses? String propURI = ((Restriction) superClass).getOnProperty(); MergedRestriction existing = (MergedRestriction) propRestriction .get(propURI); if (existing == null) { existing = new MergedRestriction((Restriction) superClass); propRestriction.put(propURI, existing); } else existing.addRestriction((Restriction) superClass); | 163 | 
| should we inherit all properties from superclasses | 181 | 
| org.universAAL.middleware.rdf.Resource | Line | 
| make this configurable | 600 | 
| check if fromIndex > propPath.length (otherwise results in infinite loop) | 1115 | 
| org.universAAL.middleware.rdf.UnmodifiableResource | Line | 
| correct? | 162 | 
| org.universAAL.middleware.serialization.turtle.TurtleUtil | Line | 
| not all double quotes need to be escaped. It suffices to encode the ones that form sequences of 3 or more double quotes, and the ones at the end of a string. | 132 | 
| org.universAAL.middleware.serialization.turtle.TurtleWriter | Line | 
| should we handle this differently? | 897 | 
| org.universAAL.middleware.service.ServiceCaller | Line | 
| AccessControl | 137 | 
| AccessControl | 229 | 
| org.universAAL.middleware.service.impl.ServiceBusImpl | Line | 
| Auto-generated method stub | 177 | 
| Auto-generated method stub | 206 | 
| Auto-generated method stub | 287 | 
| org.universAAL.middleware.service.impl.ServiceMatcher | Line | 
| change the log message to match class- and method name. BUT: update also the references, e.g. in Log Monitor Tool | 42 | 
| the better solution is to allow complex output bindings that state how the outputs are filtered by specifying a chain of restrictions (in addition to simple output bindings that only specify the corresponding property path) int expectedSize = context.size(); if (subsetService.getProfile() != null) { int i = subsetService.getNumberOfValueRestrictions(); int j = subsetService.getProfile().getNumberOfMandatoryInputs(); if (i < j) j = i; expectedSize += j; } | 169 | 
| org.universAAL.middleware.service.impl.ServiceStrategy | Line | 
| add a log entry for checking if the above assumption is true | 1076 | 
| handle somehow, e.g. send an empty/error-response | 1145 | 
| make this configurable | 1204 | 
| is the above strategy ok? The issues is: is the length of messages a good criteria? | 1277 | 
| We need to have some kind of management here or at least a timeout (but what then? Denote itself as coordinator until find the "real" one?)! | 1739 | 
| org.universAAL.middleware.service.owl.ServiceBusOntology | Line | 
| Restrictions? | 85 | 
| org.universAAL.middleware.service.owl.UserInterfaceService | Line | 
| check the methods, especially getUIServiceInfo; there seems to be something wrong (using MY_URI as property path??) | 42 | 
| org.universAAL.middleware.service.owls.process.ProcessResult | Line | 
| duplicate code with setProperty | 193 | 
| duplicate code with toResult | 243 | 
| org.universAAL.middleware.service.test.util.ServiceBusTestCase | Line | 
| put the setUp()-method in a junit.extensions.TestSetup and add a tearDown()-method so that it is possible to have multiple subclasses and to allow a reset() in a new setUp()-method so that not every untit test has to call it. | 74 | 
| org.universAAL.middleware.shell.universAAL.osgi.ConfigChangeCommand | Line | 
| select menu | 73 | 
| set all the rest of possibilities | 134 | 
| org.universAAL.middleware.shell.universAAL.osgi.ConfigPullCommand | Line | 
| select menu | 69 | 
| org.universAAL.middleware.shell.universAAL.osgi.ConfigPushCommand | Line | 
| select menu | 70 | 
| org.universAAL.middleware.shell.universAAL.osgi.TenantCommand | Line | 
| Auto-generated method stub | 35 | 
| org.universAAL.middleware.shell.universAAL.osgi.completers.ConfigEntityCompleter | Line | 
| Auto-generated method stub | 45 | 
| org.universAAL.middleware.shell.universAAL.osgi.completers.ConfigFileCompleter | Line | 
| Auto-generated constructor stub | 30 | 
| org.universAAL.middleware.tracker.impl.BusMemberRegistryImpl | Line | 
| synchronize map and listener for concurrent access | 50 | 
| org.universAAL.middleware.ui.impl.UIBusImpl | Line | 
| Auto-generated method stub | 320 | 
| org.universAAL.middleware.ui.impl.UIStrategyHandler | Line | 
| send data to DM? | 383 | 
| override newRegistration() and try to reallocate pending (and not running) dialogs | 701 | 
| org.universAAL.middleware.ui.impl.generic.CallBasedStrategy | Line | 
| monitor peers and abort all synchronous call to a peer when this disconnects form the space. | 72 | 
| org.universAAL.middleware.ui.impl.generic.CoordinatedStrategy | Line | 
| give a notification? so coordination may be reattempted. | 349 | 
| org.universAAL.middleware.ui.rdf.Repeat | Line | 
| add a log entry! | 518 | 
| org.universAAL.middleware.util.ResourceUtil | Line | 
| data values (e.g. int) | 97 | 
| org.universAAL.middleware.xsd.NonNegativeInteger | Line | 
| if the value is too big, use BigInteger | 57 | 
| org.universAAL.tests.MessagesTest | Line | 
| Auto-generated catch block | 105 |