[jira] [Commented] (WICKET-5976) Improve the documentation of FeedbackMessages first(int level)

["ASF subversion and git services (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/WICKET-5976?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14733160#comment-14733160 ] 

ASF subversion and git services commented on WICKET-5976:
---------------------------------------------------------

Commit ecf29185edbb01d0d1692a17991403b112cf9fba in wicket's branch refs/heads/wicket-6.x from [~mgrigorov]
[ https://git-wip-us.apache.org/repos/asf?p=wicket.git;h=ecf2918 ]

WICKET-5976 Improve the documentation of FeedbackMessages first(int level)


> Improve the documentation of FeedbackMessages first(int level)
> --------------------------------------------------------------
>
>                 Key: WICKET-5976
>                 URL: https://issues.apache.org/jira/browse/WICKET-5976
>             Project: Wicket
>          Issue Type: Improvement
>          Components: wicket
>    Affects Versions: 7.0.0
>            Reporter: Robert Bain
>            Priority: Minor
>   Original Estimate: 3h
>  Remaining Estimate: 3h
>
> FeedbackMessages method public final FeedbackMessage first(int level) has documentation specifying that it "Retrieves the first message matching the specified level" when in fact it calls FeedbackMessage's isLevel(int level) which "Returns whether this level is greater than or equal to the given level." which is also poorly named as it doesn't return whether the FeedbackMessage is of the specified level; it returns if it's of the specified level or more severe.
> In my opinion both the current functionality and the functionality to return only those messages of a specified level are desirable. A minimal solution is an update to the documentation of FeedbackMessage's first(int level) method but I think the methods should be renamed to say what they actually do.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)