Skip to content

Conversation

@deralmas
Copy link

Fix StorageController.Storage null check on StorageController.LoadAsync().
Before this commit it null checked a Task with a result of StorageController.Storage,
which always returns a non null object, instead of checking StorageController.Storage itself.

Fix cache reading from UTF-8 to UTF-16 as
FileInfo.OpenText creates a FileStream encoded in UTF-8.

Stop StorageController from loading its storage on instantiation as it isn't awaitable,
rendering it useless on first access and requiring to call it externally, essentially calling it twice.

Fix StorageController.Storage check.

Fix cache reading from UTF-8 to UTF-16.

Stop StorageController from loading its storage on instantiation as it isn't awaitable,
rendering it useless on first access and requiring to call it externally, essentially calling it twice.
@codecov
Copy link

codecov bot commented Aug 19, 2019

Codecov Report

Merging #319 into master will decrease coverage by 2.3%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #319      +/-   ##
==========================================
- Coverage   58.77%   56.47%   -2.31%     
==========================================
  Files          91       91              
  Lines        6128     6128              
  Branches        0     1011    +1011     
==========================================
- Hits         3602     3461     -141     
+ Misses       2526     2488      -38     
- Partials        0      179     +179
Impacted Files Coverage Δ
Parse/Internal/Utilities/StorageManager.cs 75% <100%> (+6.03%) ⬆️
...e/Internal/Storage/Controller/StorageController.cs 50% <100%> (+24.41%) ⬆️
Parse/Internal/Push/State/MutablePushState.cs 42.5% <0%> (-45%) ⬇️
Parse/Internal/Push/Coder/ParsePushEncoder.cs 61.11% <0%> (-13.89%) ⬇️
Parse/Public/ParseClient.cs 63.41% <0%> (-10.66%) ⬇️
Parse/Internal/Command/ParseCommand.cs 85.1% <0%> (-10.64%) ⬇️
Parse/Internal/Operation/ParseRemoveOperation.cs 36.84% <0%> (-10.53%) ⬇️
Parse/Public/ParseQuery.cs 14.35% <0%> (-8.72%) ⬇️
Parse/Public/ParseACL.cs 77.38% <0%> (-8.34%) ⬇️
Parse/Internal/Operation/ParseFieldOperations.cs 75% <0%> (-8.34%) ⬇️
... and 29 more

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 077b48e...121fde8. Read the comment docs.

@deralmas
Copy link
Author

deralmas commented Aug 20, 2019

Oh wait, TobiasPott already fixed this better in one of his forks . Now i don't know what to do, should i close this?

@deralmas deralmas closed this Aug 20, 2019
@deralmas deralmas reopened this Aug 20, 2019
@deralmas
Copy link
Author

well, i think i'll close it then.

@deralmas deralmas closed this Aug 21, 2019
@TobiasPott
Copy link
Member

Hi @Riteo I'm sorry that nothing happened on this issue for the last days.

Unfortunately I'm quite busy at work at the time but I have some maintaining tasks for this repo on my todo for the next week which includes performing the pull request of my changes to the StorageController.LoadAsync method.

However thank you for your PR which brought my changes back to my mind and bumping their priority on my personal todo list ;)

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.

2 participants