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

[Jon Daley]

 	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