Uploaded image for project: 'Change Request Application'
  1. Change Request Application
  2. CRAPP-365

Velocity error preventing to access a change request

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • 1.17
    • 1.16
    • discussion
    • None
    • Unknown
    • N/A
    • N/A

    Description

      Reproduction steps:

      • Create a page named "Foo"
      • Create another page with a link to Foo and an include macro referencing Foo
      • Edit that page and create a change request just before the inserting link
      • Add a new comment in the diff on the include macro (the 3 diff lines should include both the include and the link)
      • Rename the page Foo
      • Check back the change request

      Expected result:

      • The change request should be displayed fine and the comment should still be visible

      Obtained result:

      • The change request isn't visible anymore and a velocity error is displayed

      Apparently a change request in xwiki.org is not visible anymore, a velocity error is displayed, with an error.

      The root cause seems to be:

      Caused by: com.fasterxml.jackson.core.JsonParseException: Unexpected character ('\' (code 92)): was expecting comma to separate Object entries
       at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 285]
       at com.fasterxml.jackson.core.JsonParser._constructReadException(JsonParser.java:2660)
       at com.fasterxml.jackson.core.base.ParserMinimalBase._reportUnexpectedChar(ParserMinimalBase.java:741)
       at com.fasterxml.jackson.core.json.ReaderBasedJsonParser._skipComma(ReaderBasedJsonParser.java:2429)
       at com.fasterxml.jackson.core.json.ReaderBasedJsonParser.nextFieldName(ReaderBasedJsonParser.java:924)
       at com.fasterxml.jackson.databind.deser.std.BaseNodeDeserializer._deserializeContainerNoRecursion(JsonNodeDeserializer.java:539)
       at com.fasterxml.jackson.databind.deser.std.JsonNodeDeserializer.deserialize(JsonNodeDeserializer.java:104)
       at com.fasterxml.jackson.databind.deser.std.JsonNodeDeserializer.deserialize(JsonNodeDeserializer.java:25)
       at com.fasterxml.jackson.databind.deser.DefaultDeserializationContext.readRootValue(DefaultDeserializationContext.java:342)
       at com.fasterxml.jackson.databind.ObjectMapper._readValue(ObjectMapper.java:4893)
       at com.fasterxml.jackson.databind.ObjectMapper.readTree(ObjectMapper.java:3146)
       at org.xwiki.contrib.changerequest.discussions.internal.ChangeRequestDiscussionDiffUtils$UnifiedDiffBlockDeserializer.deserialize(ChangeRequestDiscussionDiffUtils.java:78)
       at org.xwiki.contrib.changerequest.discussions.internal.ChangeRequestDiscussionDiffUtils$UnifiedDiffBlockDeserializer.deserialize(ChangeRequestDiscussionDiffUtils.java:69)
       at com.fasterxml.jackson.databind.deser.DefaultDeserializationContext.readRootValue(DefaultDeserializationContext.java:342)
       at com.fasterxml.jackson.databind.ObjectMapper._readMapAndClose(ObjectMapper.java:4917)
       at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:3860)
       at com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:3828)
       at org.xwiki.contrib.changerequest.discussions.internal.ChangeRequestDiscussionDiffUtils.deserialize(ChangeRequestDiscussionDiffUtils.java:200)
       at org.xwiki.contrib.changerequest.discussions.internal.DefaultChangeRequestDiscussionService.getDiffBlockMetadata(DefaultChangeRequestDiscussionService.java:380)
      

      Attachments

        Issue Links

          Activity

            People

              surli Simon Urli
              surli Simon Urli
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: