From: Nick J. <nk...@ia...> - 2003-04-25 19:20:06
|
On Fri, Apr 25, 2003 at 12:07:58PM -0700, Rob Helmer wrote: > On Fri, Apr 25, 2003 at 10:51:59AM -0700, Bryce Harrington wrote: > > > For some reason that I could not determine, the javascript for one of > > the tests in repository.js always fails. This is the > > isEmpty(document.repository.repo_root.value) validation command. The > > other validation commands work fine. I tried it on Konqueror and > > Netscape 4.x. I ended up just commenting it out. > > I only test SW with IE and Mozilla, last I checked it worked in > Konq but it could use testing. I'll try to reproduce this. Hmm, I didn't catch this in the first email. Bryce, if it's possible to use a fairly recent Mozilla, at least for cross-testing purposes to find the problem, that would be great. > > I was getting errors when trying to update user info and it wasn't > > "setting". On line 2977 it sets the user params via: > > > > $user->{'personal'}->{"$param"} = param("$param") || ''; > > > > However there was no $user->{'personal'} value in the $user hash at this > > point so it failed. To fix this I added above the loop: > > > > $user->{personal} = { }; > > Hmm... haven't seen this before. My guess is the hash structure did not exist because it wasn't reading the config file correctly, if at all. > > create_repo() always seems to return success, even when it actually > > fails. Also, the way it writes the XML files appears to be in error. > > I can add a new repos but when I look at the list of repositories, > > instead of showing a single repos with the name I gave, it shows > > repositories with each of the fieldnames of a repository. I suspect > > this issue may be related to the 'using hash as a reference' issue > > above. > > Hmm.. I think the create_repo bug is valid. It does seem to create > the repo whether or not the checkout dir exists. Yeah I remember mentioning this before, is there a bug filed so we don't forget again? > Also, right now there's no automatic checkin, because SandWeb isn't > really a CM system so much as a CVS client. It's automatic, you just have to manualy activate the automatic checkin function each time. :) - Nick |