ExperimentInfo should use Run object start time call

[Michael Reuter]

The ExperimentInfo::getWorkspaceStartDate function should use Run::startTime as this checks both the run_start and start_time properties. Not all files insert a run_start property.

[Michael Reuter]

Refs #6246. Using Run::startTime call.

Changeset: 884d5e44751c40f245f9161991d6f0101e656cb6

[Michael Reuter]

Refs #6246. Using Run::startTime call.

Changeset: 884d5e44751c40f245f9161991d6f0101e656cb6

[Karl Palmen]

Looking at today's code from Master I see

  //---------------------------------------------------------------------------------------
  /** Return workspace start date as an ISO 8601 string. If this info not stored in workspace the
  *   method returns current date.
  *
  *  @return workspace start date as a string
  */
  std::string ExperimentInfo::getWorkspaceStartDate()
  {
    std::string date;
    try
    {
      date = m_run->startTime().toISO8601String();
    }
    catch (std::runtime_error &)
    {
      g_log.information("run_start/start_time not stored in workspace. Default to current date.");
      date = Kernel::DateAndTime::getCurrentTime().toISO8601String();
    }
    return date;
  }

which does the startTime method of m_run, which according to the header file is a pointer to a Run object.

[Stuart Campbell]

This ticket has been transferred to github issue ​7092