Flag::toArray() is no longer needed.

This is just a little housekeeping - don't mind me.

I am just answering the @todo in Flag::toArray()

To put the method into natural language.

CALL the parent method and THEN ensure that a SPECIAL subset of properties go through their respective magic getters.

Now that we have a proper schema for Flag.... I think this is redundant. As an experiment I am removing the method and letting the parent method function on its own.

  public function toArray() {
    // @todo Do we need Flag::toArray() any longer?
    $properties = parent::toArray();
    $names = [
      'flag_type',
      'link_type',
      'flagTypeConfig',
      'linkTypeConfig',
    ];

    foreach ($names as $name) {
      $properties[$name] = $this->get($name);
    }

    return $properties;
  }

As a minor side issue ,, as I touch files I want to remove the now redundant @file tags.