Uploaded image for project: 'MyFaces Extensions Validator'
  1. MyFaces Extensions Validator
  2. EXTVAL-53

adjust the package structure in view of the new storage mechanism

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Wish
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.2.1, 1.2.2, 1.1.1, 1.1.2
    • 1.2.3, 2.0.3, 1.1.3
    • Property Validation
    • None

    Description

      first of all: this change is not final until the next official release.
      everybody is allowed to express a veto!

      the classes of the new storage mechanism (see EXTVAL-49) are in the *.storage package
      some internal classes were moved to this new package.

      org.apache.myfaces.extensions.validator.crossval.CrossValidationStorage
      org.apache.myfaces.extensions.validator.crossval.CrossValidationStorageEntry
      are part of a frequently used api.

      to be consistent with the new *.storage package they should be moved to:
      org.apache.myfaces.extensions.validator.crossval.storage.CrossValidationStorage
      org.apache.myfaces.extensions.validator.crossval.storage.CrossValidationStorageEntry

      if you already implemented custom cross-validation-strategies, you just have to fix the package names in your import statements.

      please inform the community, if you have any problems with that.
      this change will be part of milestone 2. please inform us as soon as possible.
      basically you have time until the last milestone (it will be announced explicitly).
      since it's a simple refactoring step it isn't a problem to revert this change at any point very easily

      Attachments

        Activity

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

          People

            gpetracek Gerhard Petracek
            gpetracek Gerhard Petracek
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment