Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: WEB-1.2
    • Fix Version/s: WEB-2.0
    • Component/s: Web
    • Labels:
      None
    • Number of attachments :
      0

      Activity

      Hide
      Dinesh Bolkensteyn added a comment -

      Hi Matthijs,

      Can you confirm that the goal of this ticket is simply to not create a violation on:

      <!DOCTYPE html>   <!-- Compliant -->
      

      Is this the only HTML5 doctype you were thinking about?

      Thank you!

      Show
      Dinesh Bolkensteyn added a comment - Hi Matthijs, Can you confirm that the goal of this ticket is simply to not create a violation on: <!DOCTYPE html> <!-- Compliant --> Is this the only HTML5 doctype you were thinking about? Thank you!
      Hide
      Dinesh Bolkensteyn added a comment - - edited

      In fact I am going to remove this rule altogether:

      We now have a new rule which checks that the document has a DOCTYPE declaration: SONARPLUGINS-2995

      If we remove that part from this rule, then it will only verify that DOCTYPE do have a DTD.

      But if we want to allow HTML5, we must remove that DTD presence requirement.

      Hence, this rule should be removed.

      Show
      Dinesh Bolkensteyn added a comment - - edited In fact I am going to remove this rule altogether: We now have a new rule which checks that the document has a DOCTYPE declaration: SONARPLUGINS-2995 If we remove that part from this rule, then it will only verify that DOCTYPE do have a DTD. But if we want to allow HTML5, we must remove that DTD presence requirement. Hence, this rule should be removed.
      Hide
      Dinesh Bolkensteyn added a comment -

      Done

      Show
      Dinesh Bolkensteyn added a comment - Done
      Hide
      Freddy Mallet added a comment -

      Manually tested !

      Show
      Freddy Mallet added a comment - Manually tested !

        People

        • Assignee:
          Dinesh Bolkensteyn
          Reporter:
          Matthijs Galesloot
        • Votes:
          0 Vote for this issue
          Watchers:
          3 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved: