[Phpwebgallery-cvs] r1075 - in /trunk: admin/include/functions_upgrade.php include/common.inc.php include/functions_user.inc.php upgrade.php upgrade_feed.php

2006-03-10 Thread Ruben
Author: rub Date: Fri Mar 10 21:17:18 2006 New Revision: 1075 URL: http://svn.gna.org/viewcvs/phpwebgallery?rev=1075&view=rev Log: Step 4 improvement issue 301: o Correction lock categories and is_admin (functions with parameters now) o Cannot use check_status with upgrade.php and upgrade_

Re: [Phpwebgallery-cvs] r1075 - in /trunk: admin/include/functions_upgrade.php include/common.inc.php include/functions_user.inc.php upgrade.php upgrade_feed.php

2006-03-10 Thread Pierrick LE GALL
Hi Ruben, Ruben <[EMAIL PROTECTED]> writes: > o upgrade_feed.php is showed only for administrator user I do not agree with this. upgrade_feed.php is used only on builds and among developers, not on stable branches. Imagine an upgrade that modify the authentication procedure... You must see upg

Re: [Phpwebgallery-cvs] r1075 - in /trunk: admin/include/functions_upgrade.php include/common.inc.php include/functions_user.inc.php upgrade.php upgrade_feed.php

2006-03-10 Thread Ruben
Hi, upgrade_feed only on builds and among developper? I had not understood when I had readed http://phpwebgallery.net/doc/doku.php?id=en:dev. (ยง Database upgrade). I thought that will included on stables branch! Ok for remark about modify the authentification procedure. But this problem must a

[Phpwebgallery-cvs] r1076 - /trunk/include/common.inc.php

2006-03-10 Thread Ruben
Author: rub Date: Fri Mar 10 23:53:29 2006 New Revision: 1076 URL: http://svn.gna.org/viewcvs/phpwebgallery?rev=1076&view=rev Log: Step 5 improvement issue 301: o upgrade_feed.php is showed only for administrator user This functionality is removed following mail of Pierrick LE GALL Modifi

Re: [Phpwebgallery-cvs] r1075 - in /trunk: admin/include/functions_upgrade.php include/common.inc.php include/functions_user.inc.php upgrade.php upgrade_feed.php

2006-03-10 Thread Pierrick LE GALL
"Ruben" <[EMAIL PROTECTED]> writes: > Hi, > > upgrade_feed only on builds and among developper? Yes, sorry if it was not clear :-/ > [...] On authentification procedure, a directly call of > upgrade_feed.php resolve problem. You're right. > In fact, I don't like upgrade procedure with be call/

Re: [Phpwebgallery-cvs] r1075 - in /trunk: admin/include/functions_upgrade.php include/common.inc.php include/functions_user.inc.php upgrade.php upgrade_feed.php

2006-03-10 Thread Ruben
- Original Message - > If you consider upgrade_feed.php is not available in stable releases, > does it change your opinion? I do not know THE truth, I just didn't > designed upgrade_feed.php procedure to be only used by admins. Yes, I understand your philosophy now! > Else, the test of P

Re: [Phpwebgallery-cvs] r1075 - in /trunk: admin/include/functions_upgrade.php include/common.inc.php include/functions_user.inc.php upgrade.php upgrade_feed.php

2006-03-10 Thread Pierrick LE GALL
"Ruben" <[EMAIL PROTECTED]> writes: >>> Else, the test of PHPWG_IN_UPGRADE must be keep? >> >> IMO, check_upgrade is fine in upgrade.php but not in upgrade_feed.php. > > I prefer keep because I test on my local machine and on free server > and this check allow to control the use of > upgrade_feed.