Uploaded image for project: 'Crucible'
  1. Crucible
  2. CRUC-3281

Do not allow "Anonymous" to be assigned "Edit Review Details"

    XMLWordPrintable

Details

    • Suggestion
    • Resolution: Obsolete
    • N/A
    • None
    • None

    • Our product teams collect and evaluate feedback from a number of different sources. To learn more about how we use customer feedback in the planning process, check out our new feature policy.

    Description

      It breaks a whole lot of functionality in Crucible, and we don't handle it gracefully.

      If you do so, you get random null pointer exceptions

      e.g trying to add a patch to a review: 

      java.lang.NullPointerException
	at com.cenqua.crucible.revision.source.PatchSource.makeFileRevisionInfo(PatchSource.java:412)
	at com.cenqua.crucible.model.managers.PatchManager.makeFromRevision(PatchManager.java:60)
	at com.cenqua.crucible.model.managers.PatchManager.createFileRevisions(PatchManager.java:50)
	at com.cenqua.crucible.model.managers.PatchManager.createPatch(PatchManager.java:40)
	at com.atlassian.crucible.spi.impl.DefaultReviewService$6.doInTransaction(DefaultReviewService.java:342)
	at com.atlassian.crucible.spi.impl.DefaultReviewService$6.doInTransaction(DefaultReviewService.java:330)

      The UI will just spin if you try to add an attachment or patch to a review etc.

      Attachments

        Activity

          People

            Unassigned Unassigned
            andrew.myers Andrew Myers [Atlassian]
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: