Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Set totalCount and limitExceeded in DataMessageJson.query result #2656

Merged
merged 1 commit into from
Jul 15, 2019

Conversation

lorthirk
Copy link

When issuing a REST Data Message query, the totalCount and limitExceeded parameters were never returned. This PR fixes this issue.

Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>
@lorthirk lorthirk requested a review from Coduz July 12, 2019 15:12
@codecov
Copy link

codecov bot commented Jul 12, 2019

Codecov Report

Merging #2656 into develop will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #2656      +/-   ##
=============================================
+ Coverage       50.6%   50.62%   +0.01%     
  Complexity      2586     2586              
=============================================
  Files           1002     1002              
  Lines          28504    28504              
  Branches        2317     2317              
=============================================
+ Hits           14424    14429       +5     
+ Misses         13078    13073       -5     
  Partials        1002     1002
Impacted Files Coverage Δ Complexity Δ
...plugin/authentication/UserAuthenticationLogic.java 74.79% <0%> (+1.62%) 0% <0%> (ø) ⬇️
...pse/kapua/commons/security/KapuaSecurityUtils.java 86.2% <0%> (+10.34%) 0% <0%> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dee3ff5...823ab2f. Read the comment docs.

@Coduz Coduz added the Bug This is a bug or an unexpected behaviour. Fix it! label Jul 15, 2019
@Coduz Coduz merged commit 8b74499 into eclipse:develop Jul 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug or an unexpected behaviour. Fix it!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants