Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-48960

Make use of rel="nofollow" to stop crawlers going to invalid pages

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Enable the device detection setting in site settings
      2. Have a different theme for mobile devices
      3. Use a mobile device and ensure that you can follow the 'Switch to the standard theme' link and the theme switches
      4. BONUS: Use a clever mobile linked web inspector or user agent spoofing to verify the switchdevice link has rel="nofollow"
      5. Go to the site calendar and Hide global events and ensure it works
      6. Use the web inspector to ensure that the hide/show links have rel="nofollow"
      Show
      Enable the device detection setting in site settings Have a different theme for mobile devices Use a mobile device and ensure that you can follow the 'Switch to the standard theme' link and the theme switches BONUS: Use a clever mobile linked web inspector or user agent spoofing to verify the switchdevice link has rel="nofollow" Go to the site calendar and Hide global events and ensure it works Use the web inspector to ensure that the hide/show links have rel="nofollow"
    • Affected Branches:
      MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE
    • Pull Master Branch:
      MDL-48960-master

      Description

      On a few of our community sites we have logs spammed with:

      Default exception handler: Incorrect sesskey submitted, form not accepted!

      This comes from things like:
      /theme/switchdevice.php (big one - displayed on all pages when device detection is enabled)
      /calendar/set.php

      You can use nofollow to stop crawlers:
      https://support.google.com/webmasters/answer/96569?hl=en

      We could fix that specific switchdevice link - or maybe even do something clever in htmlwriter..

        Attachments

          Activity

            People

            Assignee:
            poltawski Dan Poltawski
            Reporter:
            poltawski Dan Poltawski
            Peer reviewer:
            Frédéric Massart
            Integrator:
            David Monllaó
            Tester:
            Zachary Durber
            Participants:
            Component watchers:
            Bas Brands
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              9/Mar/15