Re: was: How to add a module?

Peter Cruickshank <list-UnZd7gj/[email protected]> Thu, 6 Nov 2003 16:01:19 +0000
Newsgroups gmane.comp.cms.phpslash.devel
Message-ID <[email protected]>
On Thursday 06 Nov 2003 3:11 pm, Joe Stewart wrote:
> On Wed, Nov 05, 2003 at 10:27:54PM +0000, Peter Cruickshank wrote:
...

> > I'd miss-spelled the template name 'contentonly' as 'contentsonly'.
> >
> > I guess it took so long to pin down the answer because there have been so
> > many changes I wasnt sure where to start looking.
> >
> > I have to ask tho.... is there any way to get an error message generated,
> > rather than the page just silently dying? If the template class error had
> > been shown, I'd have been able to get some productive work done today.
> > (Or if I was less careless, I know...)
>
> I'd like that too.  There may be some commented out, but I've had some
> problems with errors in the template class not being very forgiving.
> Yours should have simply shown the applicable fancybox.tpl.  Seems a bug
> to me.

That's true... I'll keep it in mind to look at. Not sure I have the time right 
now though.

...

> > PS It truly is fascinating how many times login.php seems to instantiate
> > slashBD...
>
> Yeah.  Did Ian get the queries cleaned up some?  I think most people
> won't notice a slight delay while logging in.  After logging in the array
> is in the session.

Realised what I was seeing (I think) was the block engine going through and 
refreshing all the cached blocks.

I think it would be useful at some point to turn on the debug in pslNew, and 
see if objects are being created sensibly. Seemed to me there's a lot of 
duplicate instantiation going on, and not just of slashDB (tho that's the one 
with the big implications).

Still, this has implications for how the basic block engine works - maybe not 
for now... I think it is due a revist tho - it's been with us since I started 
being interested in this project, way back in '00. 

One thought for example - I think it would make things run smoother if column 
was moved from block_options to being it's own field in psl_block - column 
has got a lot more central to PSL's operations over the years)

P


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?   SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/