Page 1 of 1

Popfetcher Plugin does not show replies as comments

Posted: Wed Sep 04, 2019 9:34 pm
by aschlemmer
Hi all!
After updating s9y to 2.3.1, Popfetchet is not working like the 2.0.1 version which has showed replies as comments – that is our core idea of the blog :-(

Code: Select all

Detected comment/reply instead of blog entry. (schnellze.it lebt, #2594)bool(false) 
Could not save comment.
What has changed and why? Please help me ...

Additional question: Also as a part of our blog, we need to restrict the senders to more than one - we are two :-) - is there a way to reestablish this? Separating with comma or semicolon was not successful.

Thanks for every help ... I'm sure that at least Garvin remembers our project which we started around 2009 or so.
Kind regards and: long live s9y!
Achim

Re: Popfetcher Plugin does not show replies as comments

Posted: Thu Sep 05, 2019 12:13 pm
by garvinhicking
Hm, maybe something in the DB structure changed. Reading the code the only way the workComment() method can return false is if:

A.) The entry itself does not allow comments or the database retruns errors, please try to run thie SQL query on your database if possible:

Code: Select all

SELECT id, allow_comments, moderate_comments, last_modified, timestamp, title FROM serendipity_entries WHERE id = 2594
B.) The entry query itself does not resolve properly, maybe because of a 'showFutureEntries' configuration problem, or a misconfigured server time (is that enabled or disabled in your blog?) Also, missing associated authors could be a problem. Please run this query:

Code: Select all

SELECT a.email, e.title, a.mail_comments, a.mail_trackbacks
                     FROM serendipity_entries e, serendipity_authors a
                     WHERE e.id  = 2594
                       AND e.isdraft = 'false'
                       AND e.authorid = a.authorid
HTH,
Garvin

Re: Popfetcher Plugin does not show replies as comments

Posted: Thu Sep 05, 2019 8:51 pm
by aschlemmer
Thanks, Garvin!
Future entries have been disabled although I enabled them now and the comment still doesn't show up.

A) result of first query:

Code: Select all

id	allow_comments	moderate_comments	last_modified	timestamp	title
2594	 	true	1567624727	1567624727	schnellze.it lebt
B) second query:

Code: Select all

email	title	mail_comments	mail_trackbacks
abc@aschlemmer.net	schnellze.it lebt	1	1
What else could I do to resolve my issue and bring the blog to its former behaviour to show replies as comments?

Additionally, how to configure it to allow two senders to blog by Popfetcher? Like I said before, comma and semicolon are not working to separate 2 email addresses.

Thanks in advance,
Achim

Re: Popfetcher Plugin does not show replies as comments

Posted: Thu Sep 05, 2019 9:06 pm
by aschlemmer
Aha! The entry wanted to be all comments to be moderated! Now it shows up. Even this option was set to 'no' in the popfetcher configuration screen. How to turn this off permanently? There is and will never be a way to comment by a form. Only two senders can comment by email there, that's what I need ...

Popfetcher though now throws out some fancy fatal errors and leaves the backend:

Code: Select all

Fatal error: Uncaught Error: Call to undefined function serendipity_purgeEntry() in serverpath/plugins/serendipity_event_popfetcher/serendipity_event_popfetcher.php:408 Stack trace: #0 serverpath/plugins/serendipity_event_popfetcher/serendipity_event_popfetcher.php(583): serendipity_event_popfetcher->workComment('2594', Array, 'NORMAL') #1 serverpath/plugins/serendipity_event_popfetcher/serendipity_event_popfetcher.php(1545): serendipity_event_popfetcher->workEntry('Re: schnellze.i...', 'Tun die Antwort...', '3', Array, NULL, Object(stdClass)) #2 serverpath/plugins/serendipity_event_popfetcher/serendipity_event_popfetcher.php(1579): serendipity_event_popfetcher->workPopfetcher(Array) #3 serverpath/include/plugin_api.inc.php(1138): serendipity_event_popfetcher->event_hook('backend_sidebar...', Object(serendipity_property_bag), Array, NULL) #4 serverpath/serendipity_admin.php(198): serendipity_plugin_api::hook_event('backend_sidebar...', Arr in serverpath/plugins/serendipity_event_popfetcher/serendipity_event_popfetcher.php on line 408	

Re: Popfetcher Plugin does not show replies as comments

Posted: Thu Sep 05, 2019 9:19 pm
by garvinhicking
I see.

Yeah, I think that should all be salvageable (multiple mails, remove purgeEntry call, fix default setting for moderation). I‘ll get on it next week, ok?

Regards,
Garvin

Re: Popfetcher Plugin does not show replies as comments

Posted: Thu Sep 05, 2019 9:22 pm
by aschlemmer
Thanks, Garvin! Take your time, I'm happy that you support us and this project. Have a good time until then.

Re: Popfetcher Plugin does not show replies as comments

Posted: Mon Sep 09, 2019 1:28 pm
by garvinhicking
So, I've just made a commit with version 1.48 of the plugin:

https://github.com/s9y/additional_plugi ... 4879e4b63b

In my local tests, this worked properly. Can you check it out?

Regards,
Garvin

Re: Popfetcher Plugin does not show replies as comments

Posted: Mon Sep 09, 2019 1:37 pm
by aschlemmer
successful without errors, great!
How to limit the senders to exactly 2 e-mail senders, not allowing everyone?

Thanks :-)

Re: Popfetcher Plugin does not show replies as comments

Posted: Mon Sep 09, 2019 2:14 pm
by garvinhicking
Ah sorry, the code uses "," as a separator but I documented it to be ";". Just issued an update 1.49.

Then you should be able to enter "mail1@domain.com;mail2@domain.com" for the addresses.

Re: Popfetcher Plugin does not show replies as comments

Posted: Mon Sep 09, 2019 7:15 pm
by aschlemmer
garvinhicking wrote: Mon Sep 09, 2019 2:14 pm Ah sorry, the code uses "," as a separator but I documented it to be ";". Just issued an update 1.49.

Then you should be able to enter "mail1@domain.com;mail2@domain.com" for the addresses.
now the comma "," is the working separator, the semicolon is not working - I guess that you simply mixed it up in the example.com-example above :-) ?

thank you very much for helping us!

Re: Popfetcher Plugin does not show replies as comments

Posted: Mon Sep 09, 2019 7:25 pm
by garvinhicking
No, I did fix it to use ;
You still have the old version, where the comma worked (which was not my intention)

See https://github.com/s9y/additional_plugi ... 3d0c2dea88 :)

Glad to have been of help!

Re: Popfetcher Plugin does not show replies as comments

Posted: Mon Sep 09, 2019 10:01 pm
by aschlemmer
I installed https://raw.githubusercontent.com/s9y/a ... etcher.php and it still requires a ;

what's my mistake?

Re: Popfetcher Plugin does not show replies as comments

Posted: Mon Sep 09, 2019 11:28 pm
by garvinhicking
The ; is what is proper:

$onlyfrom_parts = explode(';', $onlyfrom);

So either you mistyped and meant you need to use a comma, which is not what the code should do, or you actually are now using intended behavior :-D

Re: Popfetcher Plugin does not show replies as comments

Posted: Mon Sep 09, 2019 11:49 pm
by aschlemmer
great and sorry my confusion ...