[pLog-svn] r3518 - plog/trunk/class/view

Oscar Renalias oscar at renalias.net
Sat Jun 3 08:34:19 GMT 2006


I think so.

Just before calling the parent constructor, we should check whether  
$template exists or not and act accordingly.

On 3 Jun 2006, at 11:26, Mark Wu wrote:

> If we move these code to blogView, is that mean we need to replace the
> "_template" variable?? And re-initialize it again?
>
> Mark
>
>> -----Original Message-----
>> From: Mark Wu [mailto:markplace at gmail.com]
>> Sent: Saturday, June 03, 2006 4:25 PM
>> To: 'plog-svn at devel.lifetype.net'
>> Subject: RE: [pLog-svn] r3518 - plog/trunk/class/view
>>
>> I know, that's why I need you to confirm...
>>
>> Because I have the same feeling as you about updating the
>> blogInfo in SmartyView.
>>
>> Move to BlogView?? Maybe doable.
>>
>> I will try to fixed it according to the discussion.
>>
>> Mark
>>
>>> -----Original Message-----
>>> From: plog-svn-bounces at devel.lifetype.net
>>> [mailto:plog-svn-bounces at devel.lifetype.net] On Behalf Of Oscar
>>> Renalias
>>> Sent: Saturday, June 03, 2006 4:21 PM
>>> To: plog-svn at devel.lifetype.net
>>> Subject: Re: [pLog-svn] r3518 - plog/trunk/class/view
>>>
>>> I think this is fine as long as the template that was
>> deleted was not
>>> the default one.
>>>
>>> I haven't checked the code but I think we're not allowing to remove
>>> the default one...
>>>
>>> The only thing that I don't like too much is that we have a view
>>> making changes to a blog (!!) Logically speaking it doesn't
>> make too
>>> much sense and it's a bit confusing, specially if you think
>> that the
>>> SmartyView class is only there to load templates and show them,
>>> nothing else :)
>>>
>>> Maybe we could move this code to the constructor of BlogView?
>>>
>>> On 3 Jun 2006, at 10:31, Mark Wu wrote:
>>>
>>>> Hi Oscar:
>>>>
>>>> This commit just fixed the bug list here
>>>> http://bugs.lifetype.net/view.php?id=923
>>>>
>>>> I need your confirmation about the method is doable or not.
>>>>
>>>> If not, I will revert the change.
>>>>
>>>> Mark
>>>>
>>>>> -----Original Message-----
>>>>> From: plog-svn-bounces at devel.lifetype.net
>>>>> [mailto:plog-svn-bounces at devel.lifetype.net] On Behalf Of
>>>>> mark at devel.lifetype.net
>>>>> Sent: Saturday, June 03, 2006 3:00 PM
>>>>> To: plog-svn at devel.lifetype.net
>>>>> Subject: [pLog-svn] r3518 - plog/trunk/class/view
>>>>>
>>>>> Author: mark
>>>>> Date: 2006-06-03 06:59:49 +0000 (Sat, 03 Jun 2006) New
>>> Revision: 3518
>>>>>
>>>>> Modified:
>>>>>    plog/trunk/class/view/smartyview.class.php
>>>>> Log:
>>>>> Fixed bug http://bugs.lifetype.net/view.php?id=923.
>>>>>
>>>>> When site admin delete a site template. I don't think check the
>>>>> template used by any one is a good idea. It is very time
>> consuming
>>>>> and waste our cpu resource.
>>>>>
>>>>> The better way is when we trigger the SmartyView, we check the
>>>>> templateSet that blog used exist or not.
>>>>>
>>>>> If yes, we jusy go ahead and use it. If not, we use the
>>>>> default_template and write the information back to blogInfo.
>>>>>
>>>>> Modified: plog/trunk/class/view/smartyview.class.php
>>>>>
>> ===================================================================
>>>>> --- plog/trunk/class/view/smartyview.class.php	
>>>>> 2006-06-03 06:44:16 UTC (rev 3517)
>>>>> +++ plog/trunk/class/view/smartyview.class.php	
>>>>> 2006-06-03 06:59:49 UTC (rev 3518)
>>>>> @@ -26,6 +26,7 @@
>>>>>  	
>>>>>  		var $_cachingEnabled;
>>>>>  		var $_templateName;
>>>>> +		var $_templateSetName;
>>>>>  		var $_template;
>>>>>  		var $_data;
>>>>>  		var $_viewId;
>>>>> @@ -60,13 +61,29 @@
>>>>>
>>>>>  			// name of the tepmlate
>>>>>  			$this->_templateName = $templateName;
>>>>> +
>>>>> +			// name of the template set name
>>>>> +			$blogSettings =
>> $this->_blogInfo->getSettings();
>>>>> +			$this->_templateSetName =
>>>>> $blogSettings->getValue( 'template' );
>>>>>  			
>>>>> +			// Check the template exist or not
>>>>> +			// If not, we use the defaule_template
>>>>> and also assign the default_template back to blogInfo
>>>>> +			if ( !$this->isTemplateSetExist(
>>>>> $this->_templateSetName ) )
>>>>> +			{
>>>>> +				$config =& Config::getConfig();	
>>>>> 		
>>>>> +				$this->_templateSetName =
>>>>> $config->getValue( "default_template" );
>>>>> +				$this->_blogInfo->setTemplate(
>>>>> $this->_templateSetName );
>>>>> +
>>>>> +				include_once(
>>>>> PLOG_CLASS_PATH.'class/dao/blogs.class.php' );
>>>>> +				$blogs = new Blogs();
>>>>> +				$blogs->updateBlog(
>> $this->_blogInfo );
>>>>> +			}
>>>>> +						
>>>>>  			// get the right CachedTemplate or
>>> Template object
>>>>> -            $blogSettings = $this->_blogInfo->getSettings();	
>>>>> 		
>>>>>  			$ts = new TemplateService();	
>> 		
>>>>>  			if( $this->isCachingEnabled()) {
>>>>>  				// get a CachedTemplate object
>>>>> -				$this->_template =
>>>>> $ts->CachedTemplate( $this->_templateName,
>>> $blogSettings->getValue(
>>>>> 'template' ), $this->_blogInfo );
>>>>> +				$this->_template =
>>>>> $ts->CachedTemplate( $this->_templateName,
>>>>> +$this->_templateSetName, $this->_blogInfo );
>>>>>  				// data used to
>> calculate the view id
>>>>>  				$this->_data = $data;
>>>>>  				// and generate the
>> right cache id for it @@ -102,9 +119,23 @@
>>>>>  			
>>>>>  			return $isCached;
>>>>>  		}		
>>>>> -		
>>>>>
>>>>>  		/**
>>>>> +		 * returns true if the current template set
>>>>> does exist or flase if it does not exist
>>>>> +		 *
>>>>> +		 * @return true if template set exist
>> or false otherwise
>>>>> +		 */
>>>>> +		function isTemplateSetExist( $templateSetName )
>>>>> +		{
>>>>> +			include_once(
>>>>> PLOG_CLASS_PATH.'class/template/templatesets/templatesets.clas
>>>>> s.php' );
>>>>> +			$templateSets = new TemplateSets();
>>>>> +			if( !$templateSets->isTemplate(
>>>>> $templateSetName ) && !$templateSets->isBlogTemplate(
>>>>> $templateSetName, $this->_blogInfo->getId() ) )
>>>>> +				return false;
>>>>> +			else
>>>>> +				return true;
>>>>> +		}
>>>>> +
>>>>> +		/**
>>>>>  		 * generates a unique identifier for this view.
>>>>> The cache identifier is generated
>>>>>  		 * based on the last parameter passed to the
>>> view constructor
>>>>>  		 *
>>>>>
>>>>> _______________________________________________
>>>>> pLog-svn mailing list
>>>>> pLog-svn at devel.lifetype.net
>>>>> http://devel.lifetype.net/mailman/listinfo/plog-svn
>>>>
>>>> _______________________________________________
>>>> pLog-svn mailing list
>>>> pLog-svn at devel.lifetype.net
>>>> http://devel.lifetype.net/mailman/listinfo/plog-svn
>>>>
>>>
>>> _______________________________________________
>>> pLog-svn mailing list
>>> pLog-svn at devel.lifetype.net
>>> http://devel.lifetype.net/mailman/listinfo/plog-svn
>
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.lifetype.net
> http://devel.lifetype.net/mailman/listinfo/plog-svn
>



More information about the pLog-svn mailing list