PUBLIC - Liferay Portal Community Edition
  1. PUBLIC - Liferay Portal Community Edition
  2. LPS-37199

6.2.x Akismet Wiki Integration - Marking wiki pages as Not Spam results in console errors

    Details

    • Branch Version/s:
      6.1.x
    • Backported to Branch:
      Committed
    • Fix Priority:
      4
    • Similar Issues:
      Show 5 results 

      Description

      The error is because the Akismet API key is not registered but it still sends out data to akismet.

      Steps taken:
      1. Deploy the Akismet plugin. DO NOT get an API key just yet.
      2. Add a Page and the Wiki portlet.
      3. Click All Pages.
      4. In the Main wiki node, add a page: "Akismet Me".
      5. Click the "Akismet Me" page.
      6. Click "Mark as Spam."
      7. Admin > Site Administration > Content
      8. Click Spam Moderation
      9. Mark wiki page as Not Spam.
      10. Console errors appear.

      Expected behavior:
      There should be no console errors

        Issue Links

          Activity

          Hide
          Justin Choi added a comment -

          No Longer Reproducible through Manual Testing following the steps in the description.

          No Longer Reproducible on:
          Tomcat 7.0 + MySQL 5. Portal 6.2.x GIT ID: 40af317d613f8e84a8bb6b886910eeec6d71f671. Plugins 6.2.x EE GIT ID: 1dcc256cd3fd0d511a41a16d0670b660fb4f7ea0.

          • No more console errors.
          • Page content is not missing after approval.
          Show
          Justin Choi added a comment - No Longer Reproducible through Manual Testing following the steps in the description. No Longer Reproducible on: Tomcat 7.0 + MySQL 5. Portal 6.2.x GIT ID: 40af317d613f8e84a8bb6b886910eeec6d71f671. Plugins 6.2.x EE GIT ID: 1dcc256cd3fd0d511a41a16d0670b660fb4f7ea0. No more console errors. Page content is not missing after approval.
          Hide
          Peter Shin added a comment -

          This also occurs for marking message board posts as Not Spam. Fixed that issue in this ticket too

          Show
          Peter Shin added a comment - This also occurs for marking message board posts as Not Spam. Fixed that issue in this ticket too
          Hide
          Justin Choi added a comment -

          PASSED Manual Testing following the steps in the description.

          Per Peter, Liferay + Akismet used to try to send data even if the user did not register his API key. This condition was reflected in the console error. With the fix, data is no longer send to Akismet for automatic verification without the API key first registered.

          Fixed on:
          Tomcat 7 + MySQL 5. Portal 6.1.x GIT ID: 33edca5fc2b2ddee566d9895ba68642beb1dd4cd. Plugins 6.1.x EE GIT ID: a49de4658558605dde6b9f0c20bf224d93075136.
          Tomcat 7 + MySQL 5. Portal 6.2.x GIT ID: 249e0a08b30b1280488abec5009b1a48424e03ba. Plugins 6.2.x GIT ID: 14a24c581cb6bfcfb84526fdf8071e8f1ea8e013.

          • User can mark as "Not Spam" without console errors because Akismet data will not be sent without the API key and that content type enabled in the Akismet admin.
          Show
          Justin Choi added a comment - PASSED Manual Testing following the steps in the description. Per Peter, Liferay + Akismet used to try to send data even if the user did not register his API key. This condition was reflected in the console error. With the fix, data is no longer send to Akismet for automatic verification without the API key first registered. Fixed on: Tomcat 7 + MySQL 5. Portal 6.1.x GIT ID: 33edca5fc2b2ddee566d9895ba68642beb1dd4cd. Plugins 6.1.x EE GIT ID: a49de4658558605dde6b9f0c20bf224d93075136. Tomcat 7 + MySQL 5. Portal 6.2.x GIT ID: 249e0a08b30b1280488abec5009b1a48424e03ba. Plugins 6.2.x GIT ID: 14a24c581cb6bfcfb84526fdf8071e8f1ea8e013. User can mark as "Not Spam" without console errors because Akismet data will not be sent without the API key and that content type enabled in the Akismet admin.

            People

            • Assignee:
              Justin Choi
              Reporter:
              Justin Choi
              Recent user:
              Esther Sanz
              Participants of an Issue:
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Days since last comment:
                1 year, 32 weeks, 4 days ago

                Development

                  Structure Helper Panel