[pLog-svn] r5324 - plog/branches/lifetype-1.2/class/action

[Jon Daley]

 	Did a little more testing on the site_admin flag.  I removed it, 
but since I still have the other kinds of permissions, I can still see the 
admin area.  I removed add/update/delete permissions from myself, so now I 
can only view the global settings - and that part works - pretty cool.
 	So, maybe the site_admin flag should go away?  Though it does seem 
like it would be nice if there was an easy way for users to make users an 
admin via the database.

On Sat, 21 Apr 2007, Jon Daley wrote:

> 	Duh.  I took out the dynamic part when testing after your change.
> It looks good on the devel blog and mine. Sorry about that.
>
> 	By the way, it seems relatively hard to make someone an admin,
> from the database side - ie. what does "site_admin" do?  It doesn't appear
> to do anything.  I had to add all the additional rows for each permission
> in order to promote my user to an admin (so I could check what settings
> you were using for template_cache, etc)
>
> 	Also, I wonder if our double-running wizard problem could be
> solved by removing the id column from the users_permissions table.
> Similar to how we used to get duplicates in the config table.  My guess is
> that we don't use the id column, and the database will then enforce no
> duplicate entries, so the wizard could complete properly.  Should I make
> that change?
> 	We would have to do a similar thing as we did in the config table,
> but since this is more critical if it stays wrong, I'll probably do it in
> a safer way - the config table way had trouble with different versions of
> mysql, and some people commented it out because it didn't work on their
> blog.
> 	If I grab all of the results in php, and then erase duplicates,
> that will be cross platform capatible.  The only potential problem is if
> there are so many rows that php runs out of memory.
>
>
> On Fri, 20 Apr 2007, Oscar Renalias wrote:
>> I've added a small test case to our 1.2 test blog at http://
>> devel.lifetype.net/lifetype-1.2/index.php?blogId=1, can you see if it
>> works for you there? Just have a look at the top of the page.
>>
>> You'll probably have to register a new blog too (http user=devel,
>> password=d3v3l)
>>
>> On 20 Apr 2007, at 02:33, Jon Daley wrote:
>>
>>>  	This didn't fix the problem for me.
>>>
>>> On Thu, 19 Apr 2007, oscar at devel.lifetype.net wrote:
>>>
>>>> Author: oscar
>>>> Date: 2007-04-19 16:08:22 -0400 (Thu, 19 Apr 2007)
>>>> New Revision: 5324
>>>>
>>>> Modified:
>>>>   plog/branches/lifetype-1.2/class/action/defaultaction.class.php
>>>> Log:
>>>> Fix for issue http://bugs.lifetype.net/view.php?id=1286 --
>>>> $authuser not correctly being populated in the front page of the
>>>> blog.
>>>>
>>>> Modified: plog/branches/lifetype-1.2/class/action/
>>>> defaultaction.class.php
>>>> ===================================================================
>>>> --- plog/branches/lifetype-1.2/class/action/
>>>> defaultaction.class.php	2007-04-17 21:40:35 UTC (rev 5323)
>>>> +++ plog/branches/lifetype-1.2/class/action/
>>>> defaultaction.class.php	2007-04-19 20:08:22 UTC (rev 5324)
>>>> @@ -88,6 +88,7 @@
>>>> 			// do any work... it's already been done before and we should
>>>> "safely" assume
>>>> 			// that there hasn't been any change so far
>>>> 			if( $this->_view->isCached()) {
>>>> +				$this->setCommonData();
>>>> 				return true;
>>>> 			}
>>>>
>>>>
>>>> _______________________________________________
>>>> pLog-svn mailing list
>>>> pLog-svn at devel.lifetype.net
>>>> http://limedaley.com/mailman/listinfo/plog-svn
>>>>
>>>
>>> --
>>> Jon Daley
>>> http://jon.limedaley.com/
>>>
>>> I wish there were a knob on the TV to turn up the intelligence.
>>> There's a knob called "brightness," but that doesn't work.
>>> _______________________________________________
>>> pLog-svn mailing list
>>> pLog-svn at devel.lifetype.net
>>> http://limedaley.com/mailman/listinfo/plog-svn
>>>
>>
>> _______________________________________________
>> pLog-svn mailing list
>> pLog-svn at devel.lifetype.net
>> http://limedaley.com/mailman/listinfo/plog-svn
>>
>
> -- 
> Jon Daley
> http://jon.limedaley.com/
>
> An ignorant person is one who doesn't know what you have just found out.
> -- Will Rogers
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.lifetype.net
> http://limedaley.com/mailman/listinfo/plog-svn
>

-- 
Jon Daley
http://jon.limedaley.com/

Any concern too small to be turned into a prayer
is too small to be made into a burden.
-- Corrie Ten Boom