Re: Property import/export

On 15/12/2007, Stefan Küng <tortoisesvn@gmail.com> wrote:
> Simon Large wrote:
> > Hi Stefan,
> >
> > I love the new import feature for properties :-) Quick work!
> >
> > Now the inevitable bug reports ;-)
> >
> > 1. Initially only Import and Add are enabled, and when you select an
> > item the other buttons become enabled, but they never become disabled
> > again, even when no item is selected, so we don't know which property
> > they will act on.
> >
> > 2. Export and Remove work on all selected properties, but Save and
> > Edit should only be enabled when exactly 1 property is selected.
>
> both fixed in r11613.
>
> > 3. Export offers to save as type .svnprops, but it does not
> > automatically add that extension. Import fails without this extension.
>
> Done in r11614.
> But what do you mean with "import fails"? Do you get an error? It
> shouldn't matter what extension the file has.

Yesterday I exported a set of properties as 'fred' and when I tried to
import that same file there was an error message about wrong file
format OSLT which I put down to lack of extension. But I can't
reproduce this now (with the same version).

> > 4. Import is always non-recursive ATM. For bugtraq properties I would
> > want to apply these properties recursively.
>
> The UI would get a little bit complicated if we'd have to add some
> checkbox to import recursively (which props should be recursive and
> which ones not? the user would have to select that too).
>
> You can easily doubleclick on the imported properties to edit them,
> check the "recursive" checkbox and hit OK - that will set the property
> then recursively.

OK, that's a valid way of doing it, although it has to be done one at a time.

Another thing which occurs to me is that it would be good to be able
to see the properties in the import file before applying them,
otherwise you are doing it blindly, hoping you picked the right file.
That would mean another dialog which looks almost exactly the same as
the main property list, and that would be a good place to have a
recursive checkbox/button. I know ... more work :-(

> > 5. Tooltips on the buttons would be nice.
> Added tooltips in r11615.

Thanks,

Simon

-- 
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org