MyThirdEye

Hey again,

I did a test and yes the plugin is able to convert CSV files with over 2000 rows. However, because its such a large amount of data, wordpress rejects the content for saving in the backend. 😀 I will have to look into increasing my SQL table data size for this to work. I don’t know what the max row count is unfortunately. Also if you try to convert a large file with over 2000 rows, the conversion takes a little bit and the page stops responding, I should also add a loader for this so its not confusing 😀

Thanks again
John

Hello govsocialmedia,

Thank you for bringing this to my attention however I’m not sure why it is throwing a fatal error. This code has been the same for a few updates, its weird it would throw an error now.

Does this happen if you try to activate it again?

Can I possibly get your wordpress version? I will try to replicate the problem. You said this was freshly installed so I’m assuming you didn’t update from a previous version of the plugin which helps narrow the problem down.

In the meantime, I’ll do some testing on my end, to see if I can reproduce the problem and return with a fix if I can find one.

Thanks for using my plugin and providing feedback!
John

Hey ezp,

Thanks for trying this out for me, it definitely narrows my search down a lot. I’m also glad it wasn’t a silly mistake on my part 🙂 that would have been embarrassing, but they happen 😛 I’m going to download some EQ themes and see if all of their themes don’t like my plugin or just the Divi theme. They’re very popular theme makers, so it would suck to find out if all of their themes have errors with my plugin.

Also don’t worry about the support ticket/review thing, its my pleasure. I really do enjoy helping people out and figuring out how I can improve the plugin for all users and support/feedback is the number one way I can do that. And I’m open to feature requests, so if you have any ideas on how to improve the plugin, I’m all ears 😀

Thank you very much again for sticking through this with me and providing feedback/support. I’m happy we were able to solve the issue(somewhat) in a short time. Let me know if you have anymore issues or questions, I’d be happy to help out.

John

Oh wow that was fast 😛

I think we were typing at the same time lol, I posted my long reply a minute after yours 😛 Don’t worry about the review, I understand. Sometimes my own programs make me mad so I can only imagine what other people feel 😀

That’s a really odd situation, I’m really surprised and genuinely curious as to what may be causing this issue. You said it worked if you stack two tables on the page. This is interesting, I will definitely take another look at my code to see if I’ve done something silly.

Thanks again for using my plugin and for the feedback, this will keep me busy for a little while 😛

John

Hey there ezp,

First of all thanks for using my plugin and providing feedback! It helps me out when people come to me with problems so I can fix them for everyone using the plugin. It would have been nice if you opened the issue before the rating, but that’s ok I’m not upset.

As for your issue, I checked out the link you gave me and immediately I saw that the scripts were not being loaded in properly. This is most likely a specific case issue as I have multiple wordpress test sites that seems to be working fine. I will download you Divi theme and try it out with that to make sure its not a theme compatibility issue. To narrow my search down I’d like to ask a couple questions.

1) What version of wordpress are you running?
2) Are there any plugins on your website that may be interfering with this plugin?

For an absolute immediate fix, you can edit the jtrt_Tables > public > class-jtrt-responsive-tables-public.php file

On line 142 you see this:

$test_array = isJTRTonThePage();
		if( !empty( $test_array ) ){
			wp_enqueue_script( $this->plugin_name . "-vendor", plugin_dir_url( __FILE__ ) . '../dist/public/js/jtrt-responsive-tables-vendor-public.min.js', array( 'jquery' ), $this->version, true );
			wp_enqueue_script( $this->plugin_name, plugin_dir_url( __FILE__ ) . '../dist/public/js/jtrt-responsive-tables-public.min.js', array( 'jquery' ), $this->version, true );
		}

Simply change the if statement to something that will always equal true. for example:

if(1==1){//rest of code here}

This will load the scripts on your pages whether or not the plugin detects a table on the page. This piece of code is so the scripts are not loaded in every page of your website. If you edit the IF statement as I said, it will make sure to load in the required scripts even if it can’t find a table on the page. This isn’t the best fix, but for now it should be sufficient as I do some testing. As I said I will download the Divi theme you’re using and test it out on my test website. If there is any other information you could provide for me that may be useful such as php version for your server, I’d love to hear it.

Thanks again for using my plugin and I apologize you’ve run into some issues. For future notice its best if you create a support ticket on my github page as it will send me notifications when you reply.

Please feel free to contact me if you have any questions.
John

Hello HantuUK,

I’m glad you were able to resolve the issue, I have been quite busy so its hard to respond in a timely manner.

Do you mind me asking which theme you were using that caused the issue? I’d like to be able to do some testing to see if I can find a fix for it.

Thanks,
John