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

QML-ify the UserModel, use properties rather than setter methods #4710

Merged
merged 1 commit into from
Jul 15, 2022

Conversation

claucambra
Copy link
Collaborator

@claucambra claucambra commented Jul 8, 2022

Make more use of properties over setter methods, move Q_INVOKABLE void methods to slots

Signed-off-by: Claudio Cambra claudio.cambra@gmail.com

@claucambra claucambra self-assigned this Jul 8, 2022
@codecov
Copy link

codecov bot commented Jul 8, 2022

Codecov Report

Merging #4710 (539ba93) into master (7f40e7e) will increase coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4710      +/-   ##
==========================================
+ Coverage   56.43%   56.55%   +0.12%     
==========================================
  Files         138      138              
  Lines       17070    17070              
==========================================
+ Hits         9633     9654      +21     
+ Misses       7437     7416      -21     
Impacted Files Coverage Δ
src/libsync/discovery.cpp 84.56% <0.00%> (-0.30%) ⬇️
src/libsync/syncengine.cpp 85.74% <0.00%> (+0.54%) ⬆️
src/libsync/vfs/cfapi/cfapiwrapper.cpp 77.34% <0.00%> (+1.92%) ⬆️
src/libsync/vfs/cfapi/vfs_cfapi.cpp 86.46% <0.00%> (+2.62%) ⬆️
src/libsync/vfs/cfapi/hydrationjob.cpp 56.45% <0.00%> (+3.76%) ⬆️

Copy link
Member

@camilasan camilasan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a really nice improvement :)

@camilasan
Copy link
Member

/backport to stable-3.5

src/gui/tray/UserLine.qml Outdated Show resolved Hide resolved
src/gui/tray/usermodel.cpp Outdated Show resolved Hide resolved
src/gui/tray/usermodel.h Outdated Show resolved Hide resolved
@claucambra claucambra force-pushed the bugfix/qmlify-usermodel branch 3 times, most recently from 49e4b8e to 5401b85 Compare July 15, 2022 14:06
Signed-off-by: Claudio Cambra <claudio.cambra@gmail.com>
@claucambra claucambra force-pushed the bugfix/qmlify-usermodel branch from 5401b85 to 539ba93 Compare July 15, 2022 14:47
@nextcloud-desktop-bot
Copy link

AppImage file: Nextcloud-PR-4710-539ba93fd167c63269cc7fc7cb157cd4143e130b-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

@sonarcloud
Copy link

sonarcloud bot commented Jul 15, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants