Uploaded image for project: 'WildFly Core'
  1. WildFly Core
  2. WFCORE-5285

Utilize org.wildfly.common.Assert for Null-Checks

    XMLWordPrintable

Description

    Currently there are different approaches to have null checks of parameters and throw a NullPointerException or IllegalArgumentException. Most of them do not offer the parameter name which is null or causing a null lookup. Some of them offer the parameter name but are not I18N. Some offer both, but don't reuse the common solution.
    There is an existing solution for these cases:
    org.wildfly.common.Assert.checkNotNullParamWithNullPointerException;
    org.wildfly.common.Assert.checkNotNullParam;

    The approach is to have the same method signature, keep NullPointerException and/or IllegalArgumentException in the same way as now. Only the check and throw is changed with the mentioned methods.
    Some modules require org.wildfly.common as new dependency.

     

    To ease the pull request verification process, several pull requests (by sub-project) are provided.

    Attachments

      Activity

        People

          jmesnil1@redhat.com Jeff Mesnil
          xf01213 Boris Unckel (Inactive)
          Votes:
          0 Vote for this issue
          Watchers:
          2 Start watching this issue

          Dates

            Created:
            Updated:
            Resolved: