Jump to content
Klenkes

Comments Manager doesn't change modified date of a page

Recommended Posts

I tested all day Comments with ProCache enabled and everything would work fine except one issue:

Even though my comments field has "Quiet Save" NOT enabled, the comments Manager doesn't change the modified date on a page with the comments field. That's very inconvenient for editors because they have to save the page with the new comment in order to get the Cache refreshed. And there are several hundred pages to navigate through.

Is this by design?

I hope not because I think it wouldn't make sense, I guess... If a comment is approved shouldn't the page the comment lives on be saved?

The Comments Manager is a great module because it collects all the comments for the whole website in one place.

PW 3.0.98 | Comments 1.0.7 | Comments Manager 0.0.8

Share this post


Link to post
Share on other sites

For now I did the unthinkable... I changed a core file, and it breaks my heart 💔

Since the function isn't hookable I had to edit:
/wire/modules/Process/ProcessCommentsManager/ProcessCommentsManager.module
In function processComments I added twice:

$page_to_save = $comment->getPage();
$page_to_save->save();

Poorly done... but I don't know better.

Share this post


Link to post
Share on other sites
8 hours ago, Klenkes said:

Poorly done... but I don't know better.

Normally if you copy a core module to your /site/modules/, ProcessWire asks which of the two versions you want to use. You can the select the one in /site ..and edit it as you wish. I'm not sure if it also works with Process Modules though. I also can't remember  when the feature was introduced.

Edited by kongondo
  • Like 1

Share this post


Link to post
Share on other sites
On 11/22/2018 at 5:30 PM, kongondo said:

Normally if you copy a core module to your /site/modules/, ProcessWire asks which of the two versions you want to use. You can the select the one in /site ..and edit it as you wish. I'm not sure if it also works with Process Modules though. I also can't remember  when the feature was introduced.

Yep! Thanks. I forgot about that.
Works with Processmodules too.

Share this post


Link to post
Share on other sites
2 hours ago, Klenkes said:

Works with Processmodules too.

Good to know, thanks for confirming.

Share this post


Link to post
Share on other sites

Admin note: I'm moving this thread from the "Modules/Plugins" section to the "General Support" area of the forum. Please submit general / core / core module related questions to the General Support section instead of Modules/Plugins.

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


  • Recently Browsing   0 members

    No registered users viewing this page.

  • Similar Content

    • By _NameLess_
      Hello, 
      thanks for the help the other day. 

      I continued to configure/code my page like I want it. Now i was going to add comments. 

      I installed the Comments Manager Module under "processwire/module/edit?name=ProcessCommentsManager". 
      It got installed and I can see the menuoption under "setup", called: Comments.
      Problem: 
      It does not matter what link I press. Everyone shows the following error. Even uninstalling shows this error. 
      Page: /processwire/setup/comments/
      Error: 
      Fatal error: Class 'Comment' not found in /var/www/testsite/wire/modules/Process/ProcessCommentsManager/ProcessCommentsManager.module on line 80 I am using Processwire 2.7.2.
      Is anyone here who can help me sorting this problem out?
      Best regards
      EDIT: I just realized that this might be the wrong section of the forum. If so, please move this thread. Thanks!
×
×
  • Create New...