Fix issue 83, jsonp configuration file path computation.

[namsonx]

No description provided.

[HolQue]

Hi Son,

relative paths work now!!

But please replace:

jFile=jFile.strip()

if not re.match("^[a-zA-Z]:",jFile) and not re.match("^[\\/]",jFile):
    jFile = CString.NormalizePath(jFile)

if  not(os.path.isfile(jFile)):
    raise Exception(f"File '{jFile}' is not existing!")

self.lImportedFiles.append(jFile)
self.jsonPath = os.path.dirname(jFile)

try:
    sJsonData= self.__load_and_removeComments(os.path.abspath(jFile))

with something like:

# at first normalize:
jFile = CString.NormalizePath(jFile) # (NormalizePath does also the jFile.strip() for you ;-)

# at second check existence
if  not(os.path.isfile(jFile)):
    raise Exception(f"File '{jFile}' is not existing!")

# only if file exist, further checks makes sense:
if not re.match("^[a-zA-Z]:",jFile) and not re.match("^[\\/]",jFile):
   # further consequences if required

self.lImportedFiles.append(jFile)
self.jsonPath = os.path.dirname(jFile)

try:
    sJsonData= self.__load_and_removeComments(jFile) # jFile is already normalized, therefore os.path.abspath is not necessary

[test-fullautomation]

Hi Son,
very good.
Please consider Holgers finding and provide as soon as possible a new pull-request.,
Thank you,
Thomas

[namsonx]

Hello Holger,

Thank you for your comments!

Hello Thomas, I have just updated the pull-request.

Thank you,
Son

[HolQue]

Hi Son,

what is this:

jFile = CString.NormalizePath(jFile.strip())

?

I already mentioned several times now: NormalizePath does a strip() also. Therefore no need to do this when calling NormalizePath().

Please consider my previous comment:

# at first normalize:
jFile = CString.NormalizePath(jFile) # (NormalizePath does also the jFile.strip() for you ;-)

[namsonx]

Hello Holger,

Sorry for the mistake! I've done it as a habit.
I remove .strip()

Thank you,
Son

[test-fullautomation]

Hi @namsonx ,
what is the status of this ticket?
Thank you,
Thomas

[test-fullautomation]

Hi Son,
merged after discussion with Holger,
Thank you,
Thomas