Uploaded image for project: 'Accumulo'
  1. Accumulo
  2. ACCUMULO-4525

Replace meaningless method names

Log workAgile BoardRank to TopRank to BottomBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskLinkCloneLabels
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 1.7.3, 1.8.1, 2.0.0
    • None
    • None

    Description

      There are a few abbreviated method names reused throughout the code that are meaningless to new users. Method names such as "nk", "nf" or "ane" are not helpful when trying to determine how a class works. This seems to only occur in Tests but I think it would improve readability a lot, especially for newbies. I for one think Tests are a good place to learn how a class works and choosing brevity over clarity greater hinders the readability of a Test.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            milleruntime Michael Miller Assign to me
            milleruntime Michael Miller
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0h
              0h
              Logged:
              Time Spent - 1h 40m
              1h 40m

              Slack

                Issue deployment