From: Chad <ch...@ch...> - 2003-03-30 17:51:58
|
Some more thoughts: 1) How about being able to upload more than 1 calendar at a time, let's=20= try 5. 2) Multiple delete. Let's list all the calendars with unchecked=20 checkboxs. Whichever get highlighted by the admin get deleted. 3) Remove the javascript confirm. If Im the admin, I login, select a=20 calendar, then click delete, I dont need a 4th step to confirm, its=20 just a waste of time. 4) Username and passwords will not be stored in the config. I don't=20 want them set anywhere where someone else could get access. Anyone=20 using PHP iCalendar has some sort of ftp or webdav username and=20 password already on the system. Use them. 5) Cookies. Not sure what to do here. Personally I don't think we=20 should set a cookie, since its an added security risk. I don't think=20 people will be loging in daily to admin their calendars, if they are=20 they need a better solution than the admin page. 6) Unlink. Seems like another possible security problem. If ftp is more=20= supported, than why not use it. Unlink requires no username and=20 password. Would like to hear from other developers on this one. That=20 and it doesn't work on my development machine (Mac OS X). 6) Translations. Seems like we have a good bit of new lines, perhaps we=20= can look at this and lighten them up a bit. Please don't get discouraged by this list. Your contribution is greatly=20= appreciated and will help make PHP iCalendar better. I just need to=20 think about things overall -- Ease of use, Setup, Security, Looks,=20 Functionality -- and work with all the developers (I know they are here=20= somewhere) to find a common ground. Thanks. C On Saturday, March 29, 2003, at 05:31 PM, Mike Traum wrote: > Re: unlink error, I don't get that error on my server - maybe a server=20= > config issue? This may be a better way to delete (from php.net): > <?php > function suppr($file) { > $delete =3D @unlink($file); > if (@file_exists($file)) { > $filesys =3D eregi_replace("/","\\",$file); > $delete =3D @system("del $filesys"); > if (@file_exists($file)) { > $delete =3D @chmod ($file, 0775); > $delete =3D @unlink($file); > $delete =3D @system("del $filesys");}}} > ?> > > Re: javascript confirm, it can de disabled by setting $confirm_changes=20= > to 'no' in config.inc.php, so I don't know why it needs be removed. > > Re: coding standards, I thought I was. Which part didn't meet your=20 > standards? > > mike > > =A0Chad <ch...@ch...> wrote: > > Well, it need a bit of a polish, which I can take care of over the = next > few days. I also got an error trying to delete a calendar: > > Warning : unlink() failed (Permission denied) in > /Users/clittle/Sites/phpicalendar/admin.php on line 180. > > The coding standards should also match ours, so developers can expect > consistant structure. The javascript confim can also go away. I've > committed it for now but we'll have to work it over. > > -C > > On Friday, March 28, 2003, at 12:55 PM, Mike Traum wrote: > > > Thanks for all of the good suggestions regarding the admin control > > panel. The patch is available on the sourceforge patch page. Please > > help by testing this so=A0we can get it up to release standards = asap. > > > > thanks, > > mike > > > > > > > > > > > Do you Yahoo!? > > Yahoo! Platinum - Watch CBS' NCAA March Madness, live on your=20 > desktop! > > > ------------------------------------------------------- > This SF.net email is sponsored by: > The Definitive IT and Networking Event. Be There! > NetWorld+Interop Las Vegas 2003 -- Register today! > http://ads.sourceforge.net/cgi-bin/redirect.pl?keyn0001en > _______________________________________________ > Phpicalendar-devel mailing list > Php...@li... > https://lists.sourceforge.net/lists/listinfo/phpicalendar-devel > > > <image.tiff> > > Do you Yahoo!? > Yahoo! Platinum - Watch CBS' NCAA March Madness, live on your desktop!= |