jsdecena / laracom

Laravel FREE E-Commerce Software
https://jsdecena.github.io/laracom
1.91k stars 861 forks source link

Fixes #228 #265

Closed viktormaar closed 3 years ago

viktormaar commented 3 years ago

I don´t exactly know the intention of the sale price usage, but made a fix for the https://github.com/jsdecena/laracom/issues/228 anyways.

codecov[bot] commented 3 years ago

Codecov Report

Merging #265 (080d19c) into master (6130250) will decrease coverage by 0.06%. The diff coverage is 57.14%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #265      +/-   ##
============================================
- Coverage     79.95%   79.88%   -0.07%     
- Complexity      606      607       +1     
============================================
  Files           129      129              
  Lines          2135     2138       +3     
============================================
+ Hits           1707     1708       +1     
- Misses          428      430       +2     
Impacted Files Coverage Δ
...t/app/Shop/Orders/Repositories/OrderRepository.php 94.36% <33.33%> (-2.70%) :arrow_down:
...p/Controllers/Admin/Products/ProductController.php 94.40% <66.66%> (ø)
...ject/app/Http/Controllers/Front/CartController.php 92.30% <100.00%> (ø)

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 6130250...080d19c. Read the comment docs.