Uploaded image for project: 'Commons Text'
  1. Commons Text
  2. TEXT-58

All uppercase methods?

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Trivial
    • Resolution: Implemented
    • None
    • 1.0

    Description

      Class o.a.c.text.translate.EntityArrays contains methods names with all uppercase letters (and underscores).

      I understand that they create copies of static constants (although even that is not true since they return arrays!), but are you sure you want to release a new component that does not follow the usual convention?

      I understand these comes from LANG but isn't it the right time to fix the API?

      Attachments

        Issue Links

        relates to

        Bug - A problem which impairs or prevents the functions of the product. TEXT-59 Mutable static data

        • Major - Major loss of function.
        • Closed
        links to

        Web Link GitHub - Commons Text PR#28

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

          People

            chtompki Rob Tompkins
            erans Gilles Sadowski
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment