Merge pull request #26 from bavix/fee-fix

Fee fix

Author: rez1dent3

changelog.md

@@ -6,6 +6,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.2.2] - 2019-05-12
+### Fixed 
+- fixed fee counting. see issue #25 
 
 ## [2.2.1] - 2019-02-17
 ### Added 
@@ -199,7 +202,8 @@ The operation is now executed in the transaction and updates the new `refund` fi
 - Exceptions: AmountInvalid, BalanceIsEmpty.
 - Models: Transfer, Transaction.
 
-[Unreleased]: https://github.com/bavix/laravel-wallet/compare/2.2.1...HEAD
+[Unreleased]: https://github.com/bavix/laravel-wallet/compare/2.2.2...HEAD
+[2.2.2]: https://github.com/bavix/laravel-wallet/compare/2.2.1...2.2.2
 [2.2.1]: https://github.com/bavix/laravel-wallet/compare/2.2.0...2.2.1
 [2.2.0]: https://github.com/bavix/laravel-wallet/compare/2.1.0...2.2.0
 [2.1.0]: https://github.com/bavix/laravel-wallet/compare/2.0.1...2.1.0

src/Traits/HasWallet.php

@@ -190,7 +190,7 @@ protected function assemble(Wallet $wallet, Transaction $withdraw, Transaction $
             'from_id' => $this->getKey(),
             'to_type' => $wallet->getMorphClass(),
             'to_id' => $wallet->getKey(),
-            'fee' => $withdraw->amount - $deposit->amount,
+            'fee' => \abs($withdraw->amount) - \abs($deposit->amount),
             'uuid' => Uuid::uuid4()->toString(),
         ]);
     }