Ticket #7277 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

FileProperty should strip leading & trailing whitespace from the value

Reported by: Peter Peterson Owned by: Martyn Gigg
Priority: major Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Karl Palmen

Description (last modified by Martyn Gigg) (diff)

If you supply a file with leading or trailing whitespace (like cutting and pasting from the command line), the finder will be unable to locate the file.

Change History

comment:1 Changed 7 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 7 years ago by Martyn Gigg

  • Description modified (diff)
  • Summary changed from FileProperty should strip trailing whitespace from the value to FileProperty should strip leading & trailing whitespace from the value

comment:3 Changed 7 years ago by Martyn Gigg

Strip leading & trailing whitespace from file paths. Refs #7277

Allows for copying in extra characters by accident when finding a file.

Changeset: cddab9f57d8af8458713463c5e831ea47fa762a8

comment:4 Changed 7 years ago by Martyn Gigg

  • Status changed from accepted to verify
  • Resolution set to fixed

Branch: feature/7277_filesearch_strip_whitespace

Tester: Both the FileFinder & the Load algorithms need to be checked. I used this script below and fiddled with the spaces and either end. Please also make sure the dialog boxes still work as this will affect them too. 

file_path = "/home/dmn58364/GitHub/systemtests/Data/van_gem59378_benchmark-0.nxs "

print "Found path '%s'" % FileFinder.getFullPath(file_path)

ws = Load(Filename=file_path)

comment:5 Changed 7 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:6 Changed 7 years ago by Karl Palmen

  • Status changed from verifying to closed

comment:7 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8123

Note: See TracTickets for help on using tickets.