LoadEventNexus has properties with inconsistent naming styles

[Martyn Gigg]

Some property names, FilterByTof_Min for example, contain underscores. Our policy is for PascalCase property names.

[Martyn Gigg]

In [14965]:

Bring property names inline with standards. Fixes #3877

[Nick Draper]

tested using this script

import re

def RemoveVersion(algName):
	m = re.search('\w+', algName)
	return m.group(0)
	
def FirstCapital(algName):
	m = re.search('^[A-Z].*', algName)
	if m is None:
		return "The first letter must be a capital.\n"
	else:
		return ""
		
def IllegalCaracters(algName):
	m = re.search('[\-\_\(\)\[\]]', algName)
	if m is None:
		return ""
	else:
		return "Contains illegal characters.\n"
	

with open('c:/mantid/algnames.txt', 'r') as f:
	algNames = f.readlines()
f.closed

for algName in algNames:
	errorMessage=""
	algName = RemoveVersion(algName)
	errorMessage+=FirstCapital(algName)
	errorMessage+=IllegalCaracters(algName)
	alg=mtd.createAlgorithm(algName)
	if errorMessage <>"": print algName, errorMessage
	for prop in alg.getProperties():
		propError = ""
		propError+=FirstCapital(algName)
		propError+=IllegalCaracters(algName)
		if propError <>"": print algName, prop.name, propError

[Stuart Campbell]

This ticket has been transferred to github issue ​4724