TYPO3 Core

The source of DataMapper:: currently reads:

public function convertClassNameToTableName($className = NULL) {
  if ($className !== NULL) {
    $tableName = $this->getDataMap($className)->getTableName();
  } else {
    $tableName = strtolower($className);
  }
  return $tableName;
}

This is pointless since every non-NULL value returns the table name from a matching DataMap and every NULL value is processed with strtolower. This should be removed.

It also does not make sense to make the $className parameter optional.