Posted on: 08/06/07 11:50am
By: Phize
Dear all:
Hello, this's my first time to post a message here.
I'm Phize which is a member of Geeklog Japanese.
Now, I found some problems in Polls plugin, and I think those due to typo. of Block's name.
At first, 'polls_block' is set for Polls plugin's 'name' filed of 'gl_blocks' table in DB.
But, in /plubins/polls/functions.inc, 'poll_block' is set for creating HTML by template classes.
This typo. causes some problems.
For example, poll plugin's HTML in left/right block is created by using center block's templates(blockheader.thtml, blockfooter.thtml).
So I modifed them as the followings.
/plugins/polls/functions.inc
---
616: $title = DB_getItem( $_TABLES['blocks'], 'title', "name='polls_block'" );
630: $retval .= COM_startBlock( $title, '', COM_getBlockTemplate( 'polls_block', 'header' ));
651: $retval .= COM_endBlock( COM_getBlockTemplate( 'polls_block', 'footer' ));
---
I hope these problems are corrected in official package.
Finally,
thank you for every great work, I respect you in Geeklog.net.
Best Regards.
---------------
Phize
Re: Some typo. of Block's name in Polls plugin.
Posted on: 08/06/07 02:45pm
By: Dirk
Thanks for spotting this. The situation is actually somewhat more confusing, though.
On a fresh install of Geeklog 1.4.0 or 1.4.1, the block will be named 'polls_block'. However, when you upgrade from an older version, the block will keep its old name, which was 'poll_block' - without the 's'
We'll have to stick with one name (most likely "polls_block", since it's the Poll
s plugin after all) and clean up the upgrade process (and adopt your suggested fix).
bye, Dirk
Re: Some typo. of Block's name in Polls plugin.
Posted on: 08/07/07 05:39am
By: Phize
Hello, Dirk.
Thanks for your quick response, and new information for us.
We knew those problems come from the process of upgrading.
Then we'll fix them and release our next package in the near future.
Thanks again.
-----------------------------------------
Geeklog Japanse and Phize