Commit Graph

867 Commits

Author SHA1 Message Date
9491e5a547 Merge pull request #14954 from peppy/rename-result-response
Rename `APIRequest.Result` to `Response`
2021-10-06 10:52:33 +09:00
c23acb8f05 Merge pull request #14967 from bdach/fix-accidental-json-property-rename
Fix "most played beatmap" request breakage after property rename
2021-10-06 05:00:12 +09:00
baa8baaa1e Fix "most played beatmap" request breakage after property rename 2021-10-05 21:17:19 +02:00
12da27cde7 Add test coverage for loading process on channel join 2021-10-05 20:52:40 +02:00
b41fa41c85 Rename APIRequest.Result to Response 2021-10-05 14:28:56 +09:00
853cf6feaa Rename last remaining BeatmapInfo Beatmap usage 2021-10-04 17:35:53 +09:00
5937a93e2d Merge pull request #14949 from peppy/login-error-display
Show login failure messages on login form
2021-10-04 17:23:11 +09:00
3a0b7ba8ff Add fallback to use Message when Hint is not available 2021-10-04 16:18:55 +09:00
266b4c7124 Expose login errors from IAPIProvider and show on the login form 2021-10-04 15:40:24 +09:00
5aaafce597 Make AuthenticateWithLogin throw instead of return a bool success status 2021-10-04 15:40:00 +09:00
ec61c3c5ee Rename all remaining cases 2021-10-03 00:55:29 +09:00
d9b4fae4e5 Merge branch 'master' into chat-command 2021-09-13 13:32:44 +09:00
1b264a2dd0 make user lookup string public 2021-09-11 16:22:29 +01:00
f4b1d8b9d1 Merge pull request #14704 from peppy/modicon-imod-support
Update `LeaderboardModSelector` to avoid creating mod instances
2021-09-10 16:43:32 +09:00
309460e505 Merge pull request #14675 from ekrctb/featured-artist-markers
Add featured artist markers to beatmap listing and overlay
2021-09-10 14:30:33 +09:00
28e9329136 Update LeaderboardModSelector to avoid creating mod instances 2021-09-10 12:43:21 +09:00
76e877f160 Disable APIMod/Mod cross equality support 2021-09-10 11:24:56 +09:00
cf633973a9 Refactor exposed mod retrieval methods for better safety 2021-09-10 11:09:13 +09:00
ce6b022a90 Remove unused IMod specification from APIMod 2021-09-10 10:59:30 +09:00
2edb851008 Add ability to lookup mod from a type specification 2021-09-09 16:50:59 +09:00
4d0530ca9d Add new methods to ruleset for quicker mod lookups 2021-09-09 16:46:14 +09:00
e66d76d26e Avoid settings copy if there are no settings 2021-09-09 15:43:55 +09:00
7543f9dfb0 Add featured artist track ID online info 2021-09-08 12:21:24 +09:00
be9540e535 fix "key" having wrong url parameter name 2021-09-07 17:17:10 +01:00
b6c80f04b0 Add "featured artists" filter to beatmap search 2021-09-07 16:44:45 +09:00
1c4a3c584a Use correct lookup type to ensure username based lookups always prefer username 2021-09-06 15:04:27 +09:00
e78dc1bb4c more code quality :/ 2021-09-05 15:27:28 +01:00
e409f2dc6d add xmldoc 2021-09-05 10:42:38 +01:00
8104b15874 remove braces
Co-authored-by: Salman Ahmed <frenzibyte@gmail.com>
2021-08-30 13:23:33 +01:00
90c313e2ad add methods to get a user from their username 2021-08-29 19:19:55 +01:00
8cd972a9fe Merge branch 'master' into pinned-comment 2021-08-24 14:37:31 +09:00
da8eba9996 Return early to avoid updating state and failure count in fail cases 2021-08-20 12:11:41 +09:00
a6b7ca1a4c Ensure all request failures are correctly handled during login 2021-08-19 19:55:14 +09:00
7d7c5c06f0 Fix code formatting 2021-08-15 16:02:25 +02:00
2f9f1ba862 Add test for ChannelManager.MarkChannelAsRead 2021-08-15 15:44:23 +02:00
480d5ffa5d add pinned comment to users setter in comment bundle 2021-08-13 19:40:37 +07:00
39b13efdd5 add pinned comments property in comment bundle 2021-08-13 13:13:28 +07:00
0901333ef3 add pinned property in comment 2021-08-13 12:57:29 +07:00
556962a3d8 Add missing xmldoc comment
Co-authored-by: Bartłomiej Dach <dach.bartlomiej@gmail.com>
2021-08-03 14:50:56 +09:00
617ff40de7 Add the ability to not use MessagePack when creating a HubConnector 2021-08-02 18:50:22 +09:00
185ea776f5 Fix incorrect authorisation loss exception handling with recent changes 2021-07-30 18:11:40 +09:00
3a5324c947 Fix aborting an APIRequest potentially resulting in incorrect success 2021-07-29 13:01:52 +09:00
1ed4fdd5f5 Avoid deserialisation JSON request content when error is not present (or not relevant) 2021-07-28 20:13:40 +09:00
bb3747ffc9 Fix beatmap search requests double-escaping
Closes #14008.
2021-07-25 17:06:40 +09:00
17168b8137 Fix authentication loss not handled correctly
This handles the case where on initial API connection, the server
responds with an `Unauthorized` response. It doesn't perform this same
checking/handling on every API request, which is probably what we want
eventually.

Opting to not address the full issue because I know this is going to be
a long one (see
05c50c0f6c/osu.Game/Online/API/APIAccess.cs (L233)).
2021-07-23 19:03:19 +09:00
57e5f5575a Fix derived API request types firing success when they shouldn't
The usual case of `virtual`/`override` being dangerous when logic is
added to the base implementation. As such, I've removed this completely.
2021-07-22 14:23:27 +09:00
747c475b95 Merge pull request #13932 from peppy/fix-multiple-request-completion-states
Fix API request potentially firing failed events after completion
2021-07-21 15:11:03 +09:00
1e634d9db0 Adjust user beatmap sections on profile overlay to match web 2021-07-20 21:55:17 +02:00
80c2b1449b Fix API request potentially firing failed events after completion
Specifically, `Cancel()` calls were not thread safe. Due to a series of
events, `ListPollingComponent` could call `Cancel` from a non-update
thread, leading to a race condition where both a `Success` and `Fail`
event can be fired.

This is intended to be the simplest fix possible, locking and guarding
specifically on the callbacks. Further work could be done in the future
to improve the flow surrounding `pendingFailure`, potentially reducing
redundant work and cleaning up the code, but that's not happening here.

Closes https://github.com/ppy/osu/issues/13632.
2021-07-19 20:27:01 +09:00
e1c646b9b2 Remove redundant arguments 2021-07-05 23:52:39 +08:00