Welcome to Geeklog Thursday, May 23 2013 @ 01:23 AM EDT
|
||||||||
![]() |
Forum Index > Extensions > Plugins |
New Topic
|
Post Reply
|
evList is available now! |
|||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
find the file here.
|
||||||
|
|||||||
| Anonymous: geekdude |
|
||||||
![]() |
Too bad it won't install SQL errors... Hacked through those but it won't run either, more SQL errors
Thu Sep 27 14:08:24 2007 - 1242: Subquery returns more than 1 row. SQL in question: SELECT format FROM gl_evlist_dateformat WHERE id = (SELECT date_format FROM gl_evlist_settings) Did anyone actually test this before released |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
thanks for your 2cents, but i'm not really interested in your anonymous attitude. it has been uninstalled and installed about 22 times in the last 2 weeks without issue. I've been the only one testing and it works great.
The only bug i've found so far that hasn't been fixed is in the category management page. and it's not one that breaks the plugin, just makes creation an extra click or two. about your error: your settings table should only have one row, which means you screwed up the install and left the table before you installed it again. remove the extra row(s). |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
whoops. had a quick look and the last change i made before i tarred it up double up on the default values being installed.
here is what is was: PHP Formatted Code // Default data// Insert table name and sql to insert default data for your plugin. // This will create the block definition and set the default user preferences $DEFVALUES = array(); $DEFVALUES[] = "INSERT INTO {$_TABLES['evlist_categories']} VALUES ('','General','0'),('','Birthdays','0'),('','Seminars','0')"; $DEFVALUES[] = "INSERT INTO {$_TABLES['evlist_settings']} SET usermenu_option='1', week_begins='1'"; $DEFVALUES[] = "INSERT INTO {$_TABLES['blocks']} SET is_enabled='0', name='evlist', type='phpblock', title='{$LANG_EVLIST05[2]}', tid='all', blockorder='255', onleft='1', phpblockfn='phpblock_evlist_upcoming'"; remove the categories and settings inserts like so: PHP Formatted Code // Default data// Insert table name and sql to insert default data for your plugin. // This will create the block definition and set the default user preferences $DEFVALUES = array(); $DEFVALUES[] = "INSERT INTO {$_TABLES['blocks']} SET is_enabled='0', name='evlist', type='phpblock', title='{$LANG_EVLIST05[2]}', tid='all', blockorder='255', onleft='1', phpblockfn='phpblock_evlist_upcoming'"; |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
I've just uploaded the fixed archive. look for version 1.0.1 in the downloads section.
|
||||||
|
|||||||
| Anonymous: chump |
|
||||||
|
|
Thanks for the work; however I am geting a error on the plugin install. Error stats :1264: Out of range value adjusted for column 'id' at row 1. SQL in question:
Any ideas? PHP 5.x MySQL 5.x IIS 6 |
||||||
|
|||||||
| Anonymous: Chump |
|
||||||
|
|
Commented out the following from my.ini and restarted MySQL.
#sql-mode="STRICT_TRANS_TABLES,NO_AUTO_CREATE_USER,NO_ENGINE_SUBSTITUTION" Installed fine. |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
good to know. Thanks for that.
I was at a loss |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
k, not totally lost.
it was caused by the empty strings being inserted (default values) into the INT column of the categories table. It doesn't hurt to supply id's for those few default inserts (e.g., 1,2, and 3). Removing the default values for the categories all together also wont hurt. They're just examples anyway. So if you want to run mySQL 5 in strict mode (which really you should want to) then you can expect more of these errors. I'll have to rewrite a small number of queries to deal with it. |
||||||
|
|||||||
| Anonymous: geekdude |
|
||||||
![]() |
it has been uninstalled and installed about 22 times in the last 2 weeks without issue. I've been the only one testing and it works great. whoops. had a quick look and the last change i made before i tarred it up double up on the default values being installed. that answer my question. i would hav done the testing after the changes. no mysql5 support either, nice. |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
Quote by: geekdude that answer my question. i would hav done the testing after the changes. no mysql5 support either, nice.[/p] more anonymous attitude, nice. So go do some testing and quit complaining. nobody's forcing you to use it and you're getting it for free so I really don't understand why you're being such a jerk--though I have a theory. |
||||||
|
|||||||
| DTrumbower |
|
||||||
![]() ![]() ![]() ![]() ![]() Moderator ![]() Status: offline ![]() Registered: 01/08/03 Posts: 507 |
When trying to edit an existing event, I get the following.
PHP Formatted Code Fatal error: Call to undefined function: sec_getgroupdropdown() in /var/www/fvyha.com/geeklog14/public_html/evlist/event.php on line 690 |
||||||
|
|||||||
| Dirk |
|
||||||
![]() ![]() ![]() ![]() ![]() Admin ![]() Status: offline ![]() Registered: 01/12/02 Posts: 13027 |
That function was only introduced in Geeklog 1.4.1. So I guess that's the minimum requirement for this plugin (and should probably be checked during installation).
bye, Dirk |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
the file name should probably tip you off that 1.4.1 is required. "evlist_1.0.1_1.4.1.tar.gz".
guess I can throw a "if function_exists" check into the routine. |
||||||
|
|||||||
| Blaine |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member ![]() Status: offline ![]() Registered: 07/16/02 Posts: 1233 |
It's probably best to use this function in the install.php and test for the existance of the function before the user is allowed to install it.
function plugin_compatible_with_this_geeklog_version () { return true; } Geeklog components by PortalParts -- www.portalparts.com |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
found another minor bug: when centerblock is set to appear after featured story it actually appears at the top of the page as well as after the featured story. that kinda sucks.
It was just a misplaced curly brace--sorry. Find the following in plugin_centerblock_evlist(): PHP Formatted Code // get position from stories query to account for 'after featured' position. if ($position == 2) { if (($where == 1) && ($_CONF['showfirstasfeatured'] == 0)) { if (!empty($topic)) { $AND = " AND tid = '$topic'"; } //$result = DB_query("SELECT sid FROM {$_TABLES['stories']} WHERE featured = '1'" . $AND . " LIMIT 1"); if (DB_count($_TABLES['stories'], array('featured', 'tid'), array(1, $topic)) < 1) { // no featured story found - redefine position $position = 1; } } } and change it to this: PHP Formatted Code // get position from stories query to account for 'after featured' position. if ($position == 2) { if (($where == 1) && ($_CONF['showfirstasfeatured'] == 0)) { if (!empty($topic)) { if (DB_count($_TABLES['stories'], array('featured', 'tid'), array(1, $topic)) < 1) { // no featured story found - redefine position $position = 1; } } } } |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
a couple of date displays weren't being formatted according to the config's setting. in event.php, evlist_display_event() find the following: PHP Formatted Code $A['date_end1'] = (!empty($A['date_end1']) && $A['date_end1'] != '0000-00-00') ? date("Y-m-d", strtotime($A['date_end1']) + $timediff) : ''; $A['date_start2'] = (!empty($A['date_start2']) && $A['date_start2'] != '0000-00-00') ? date("Y-m-d", strtotime($A['date_start2']) + $timediff) : ''; $A['date_end2'] = (!empty($A['date_end2']) && $A['date_end2'] != '0000-00-00') ? date("Y-m-d", strtotime($A['date_end2']) + $timediff) : ''; and change to this: PHP Formatted Code $A['date_end1'] = (!empty($A['date_end1']) && $A['date_end1'] != '0000-00-00') ? evlist_displayFormatedDate(strtotime($A['date_end1']) + $timediff) : ''; $A['date_start2'] = (!empty($A['date_start2']) && $A['date_start2'] != '0000-00-00') ? evlist_displayFormatedDate(strtotime($A['date_start2']) + $timediff) : ''; $A['date_end2'] = (!empty($A['date_end2']) && $A['date_end2'] != '0000-00-00') ? evlist_displayFormatedDate(strtotime($A['date_end2']) + $timediff) : ''; |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
Quote by: Blaine It's probably best to use this function in the install.php and test for the existance of the function before the user is allowed to install it. apparently I did have this defined in my install.php, but never finished it. Must've been distracted. |
||||||
|
|||||||
| casper |
|
||||||
![]() ![]() ![]() ![]() ![]() Regular Poster ![]() Status: offline ![]() Registered: 02/11/04 Posts: 113 |
Hey Machinari!
Nice plugin, have tested it a bit now and found it to suit some needs :ThumbsUp: One problem though: I have url_rewrite enabled, and that does not work well in evlist for me. When clicking on an event I get redirected to the event listing instead of the details for the event, and the profile-link also gets messed up. what do I need to do solve this? |
||||||
|
|||||||
| machinari |
|
||||||
![]() ![]() ![]() ![]() ![]() Full Member Status: offline ![]() Registered: 03/22/04 Posts: 1512 |
My bad--obviously.
I used COM_buildURL() with the intention that I would support url_rewrite and I neglected to follow it through. I'll fix it up and shoot you a new file probably by tomorrow. thanks for the heads up. |
||||||
|
|||||||
| Content generated in: 2.34 seconds |
|
|
|