[Ids-devel] Album removal coming soon

[Jeff M. <jef...@vi...>]

Hi Folks,
I'm working on adding album removal to IDS. If someone has done this
all ready, please speak up!

One thing that puzzles me is the following code that checks the album
paths:
      $newalbumname =3D~ s/^[\.]+|\///g; # hax0r protection...

The end result is that ANY / is removed from the album name! That just
doesn't seem right.

I've replaced that code with the following:

      if($rmalbumname =3D~ m|\.\./|) {
      	croak "path has ../ in it"; # must be a more elegant way
      }

I've only done this when the method is rmalbum, which I added for album
removal. I guess I'm asking is this make better sense to anyone else.

My current code adds a link to an album page that says 'Remove this
album'. It then will remove all the files and the album and then display
that album's parent.


--=20
Jeff Macdonald <jef...@vi...>
My birding blog: http://www.virtualbuilder.com/archives/cat_birding.html
Into birding? Visit http://www.migratus.com