3 of 3
3
New Extension: Edit Table Plus (Table Sorting for 1.6.x!)
Posted: 12 April 2010 11:40 AM   [ Ignore ]   [ # 37 ]  
Research Assistant
Avatar
RankRankRank
Total Posts:  555
Joined  04-23-2008

Just finished watching the Adobe CS5 launch video on the website. Here’s the screenshot mate — hope it helps!

Profile
 
 
Posted: 01 May 2010 01:44 PM   [ Ignore ]   [ # 38 ]  
Lab Technician
Avatar
RankRankRankRank
Total Posts:  1156
Joined  02-02-2007

Hi,

Huge thanks for this amazing extension.

I found a couple of issues:

1) When you install this extension v1.2 and open Settings page several PHP errors are displayed:

Notice: Undefined index: sorting in /home/classic1/public_html/eesys/lib/edit_table_plus/views/settings_form.php on line 10
Notice: Undefined index: default_sort in /home/classic1/public_html/eesys/lib/edit_table_plus/views/settings_form.php on line 16
Notice: Undefined index: sort_order in /home/classic1/public_html/eesys/lib/edit_table_plus/views/settings_form.php on line 22
Notice: Undefined index: render_table in /home/classic1/public_html/eesys/lib/edit_table_plus/views/settings_form.php on line 29
Notice: Undefined index: searching in /home/classic1/public_html/eesys/lib/edit_table_plus/views/settings_form.php on line 41
Notice: Undefined index: hide_select in /home/classic1/public_html/eesys/lib/edit_table_plus/views/settings_form.php on line 47

2) This extension prevents deselecting all checkboxes by clicking topmost checkbox. (It is possible to select all checkboxes by clicking topmost one, but not deselect).

 Signature 

Full list of add-ons here

Child Categories
Browser Sniff
Category Id
Entries Number

Profile
 
 
Posted: 03 May 2010 09:43 AM   [ Ignore ]   [ # 39 ]  
Lab Assistant
Avatar
RankRank
Total Posts:  145
Joined  09-18-2008

@Laisvunas

I haven’t seen any of the PHP notices before, although I’m sure it’s because I haven’t had php’s error reporting enabled.

I’ll look and see what I can do about the checkbox. The table sorting plug creates an onClick even for the cell itself, and since the checkbox is a child of the cell, clicking on the checkbox fires the parent’s event. I’m going to try to disable sorting for that column and hopefully that will solve the issue.

Profile
 
 
Posted: 03 May 2010 11:22 AM   [ Ignore ]   [ # 40 ]  
Lab Assistant
RankRank
Total Posts:  117
Joined  10-07-2007

Nice extension, this is exactly what we were looking for!

I noticed it didn’t play well with Masked CP URL (since it’s looking for js files in the lib folder).  I moved the css,images,js folder out to the themes folder and defined a path in the code like so:

if (!defined('EDIT_TABLE_PLUS')) {
                define
('EDIT_TABLE_PLUS'$PREFS->ini('theme_folder_url'1).'edit_table_plus/'); 

      }

Profile
 
 
Posted: 03 May 2010 12:07 PM   [ Ignore ]   [ # 41 ]  
Lab Assistant
RankRank
Total Posts:  117
Joined  10-07-2007

Also, there seems to be a conflict between this ext and filter_by_author.

I added

$o $EXT->last_call

to the edit_entries_search_form function.

Profile
 
 
Posted: 03 May 2010 02:26 PM   [ Ignore ]   [ # 42 ]  
Summer Student
Total Posts:  26
Joined  09-22-2004

fyi.. I just got the same php notices as @Laisvunas. once I submitted the form for the settings they went away.

Profile
 
 
Posted: 03 May 2010 02:53 PM   [ Ignore ]   [ # 43 ]  
Lab Assistant
Avatar
RankRank
Total Posts:  145
Joined  09-18-2008

@immd, @Laisvunas
Oh…If your updating from 1.x.x to 1.2.x then you need to enable/disable the extension and reset the form settings. I mentioned it http://expressionengine.com/forums/viewthread/148089/P18/#736829 , but I’ll be sure to include this in the first post and changelog as well.

@Stephen (tmoswift)
I’ll be sure to make the appropriate updates for future releases. The edit_table_plus folder will be moving from the system/lib folder to the themes/cp_themes/ folder in future releases.

Profile
 
 
Posted: 03 May 2010 03:04 PM   [ Ignore ]   [ # 44 ]  
Summer Student
Total Posts:  26
Joined  09-22-2004

mine was a first install and not updating from previous version…

Profile
 
 
Posted: 03 May 2010 03:06 PM   [ Ignore ]   [ # 45 ]  
Lab Assistant
Avatar
RankRank
Total Posts:  145
Joined  09-18-2008

Alright, I’ll be looking into it.

Profile
 
 
Posted: 05 May 2010 09:16 AM   [ Ignore ]   [ # 46 ]  
Lab Technician
Avatar
RankRankRankRank
Total Posts:  1156
Joined  02-02-2007

Hi,

I did some hacking and fixed 3 issues:

1) PHP error notices - fixed by adding PHP error control operators.

2) Table sorting interfering with search form’s settings, as e.g. in search form you set sort order to be alphabetical, but since default table sorting is set to be by date you get list of entries ordered by date. I fixed it by introducing new option - “None” - in extension’s “Sort order” setting. If “None” is selected, then no default table sorting will occur on page load, so no interference with search form’s settings will occur.

3) Extension prevents deselecting all checkboxes by clicking topmost checkbox. I fixed it by removing functions which tablesorter javascript has bound to “Check all” table header cell.

If you like my additions, fell free to use them in official version.

File Attachments
edit_table_plus.zip  (File Size: 18KB - Downloads: 72)
 Signature 

Full list of add-ons here

Child Categories
Browser Sniff
Category Id
Entries Number

Profile
 
 
Posted: 05 May 2010 09:41 AM   [ Ignore ]   [ # 47 ]  
Lab Assistant
Avatar
RankRank
Total Posts:  145
Joined  09-18-2008

@Laisvunas

Thanks. I’ll integrate those changes into 1.2.1.

Profile
 
 
Posted: 09 June 2010 11:20 PM   [ Ignore ]   [ # 48 ]  
Research Assistant
Avatar
RankRankRank
Total Posts:  465
Joined  05-15-2007

Hi there,

Thanks for this extension. As reported above, I also got those PHP errors when first installing the plugin, but they want away after saving the options for the first.

Also mentioned above, I found that this doesn’t work with a masked control panel. Unfortunately, I couldn’t get the fix mentioned by Stephen working as I don’t really know PHP and wasn’t sure where to stick that code without getting errors.

Would be great if you included support for masked control panels in a future version. grin

Thanks.

 Signature 

 
Stephen McIver | Made by Bridge | Huddersfield & Liverpool, UK | Twitter

Profile
 
 
Posted: 09 June 2010 11:24 PM   [ Ignore ]   [ # 49 ]  
Lab Assistant
Avatar
RankRank
Total Posts:  145
Joined  09-18-2008

I’ve been awfully busy the past couple of weeks (and will be for the next couple of days), but an updated version is in the works to settle these issues.

Profile
 
 
Posted: 13 June 2010 10:36 PM   [ Ignore ]   [ # 50 ]  
Research Assistant
Avatar
RankRankRank
Total Posts:  465
Joined  05-15-2007

No worries (and no rush), and thanks for letting us know. Good to know there’s an updated version in the works.

Thanks.

 Signature 

 
Stephen McIver | Made by Bridge | Huddersfield & Liverpool, UK | Twitter

Profile
 
 
   
3 of 3
3