14507 - cache file pathnames point to wrong place

Status: VERIFIED FIXED

(CCK Graveyard :: CCK-Wizard, defect, P2)

Description

[selmer (gone)]

The pathnames we're shipping inside the cache file are fully qualified names
such as c:\cck\Configs\Default\animlogo\smallanim.gif.  On my machine, I
installed into c:\cck9.21 so these pathnames are all wrong.

It seems like the best solution to this problem is to make all pathnames that
live within our install directory into relative names.  The way to do this is to
detect the presence of a prefix and remove that prefix from the pathnames before
storing them and adding it back when retrieving them.  Sometimes this might be
wrong, but not often.

I don't think we can make them relative to the configuration's own directory
since we want configurations to point into each other (such as the Default.)
Therefore, the next best place to pick is the directory containing the CCK
Wizard executable.

Comment 1

[selmer (gone)]

A refinement to the above discussion is that we could allow globals to reference
other globals.  We need to create a global like ROOT that points at the
executable directory anyway for other reasons.  If we did this, then we could
have FOOBMP=%ROOT%\Defaults\Images\foo.bmp and that would also resolve this
problem elegantly.

To code this, simply run the variable through replaceVars() if you detect a
matched pair of '%' characters in its value.

This solution would open the problems of recursive reference and such, but I'm
not too worried about that.  Other solutions are possible...

Comment 2

[varada]

I have fixed this such that while the cache is being read - the %root% is
evaluated as the current working directory - the top level cache can then needs
to have only the %root% and it will work in any directory - the other cache
files are created by the user values and hence we are not dealing with that.
changes have to be made to the default cache to reflect %Root% wherever we need
a relative path - eg %Root%Configs\Defaults\Bitmaps\animlogo.bmp. instead of
c:\..\..\...

Comment 3

[bmartin]

Marking this Verified Fixed