chamilo-lms
chamilo-lms copied to clipboard
Lp - Add tables for user and groups subscribed to lp - refs CT#3411
Codecov Report
Merging #4333 (f70f7b5) into master (1b78b94) will decrease coverage by
0.42%
. The diff coverage isn/a
.
:exclamation: Current head f70f7b5 differs from pull request most recent head 5beb77e. Consider uploading reports for the commit 5beb77e to get more accurate results
@@ Coverage Diff @@
## master #4333 +/- ##
============================================
- Coverage 48.61% 48.19% -0.42%
+ Complexity 7939 7927 -12
============================================
Files 643 643
Lines 25820 24061 -1759
============================================
- Hits 12553 11597 -956
+ Misses 13267 12464 -803
Impacted Files | Coverage Δ | |
---|---|---|
src/CoreBundle/DataFixtures/LanguageFixtures.php | 0.00% <0.00%> (-15.39%) |
:arrow_down: |
...dle/Security/Authorization/Voter/ResourceVoter.php | 57.14% <0.00%> (-10.72%) |
:arrow_down: |
...dle/DataProvider/Extension/SocialPostExtension.php | 75.00% <0.00%> (-8.34%) |
:arrow_down: |
...eBundle/Repository/ExtraFieldOptionsRepository.php | 18.18% <0.00%> (-6.82%) |
:arrow_down: |
...rseBundle/Entity/CStudentPublicationCorrection.php | 46.66% <0.00%> (-6.28%) |
:arrow_down: |
...eBundle/Security/Authorization/Voter/UserVoter.php | 73.68% <0.00%> (-5.49%) |
:arrow_down: |
...e/Security/Authorization/Voter/SocialPostVoter.php | 61.76% <0.00%> (-4.91%) |
:arrow_down: |
...Bundle/Repository/TrackECourseAccessRepository.php | 18.18% <0.00%> (-4.90%) |
:arrow_down: |
src/CoreBundle/Serializer/UserToJsonNormalizer.php | 43.93% <0.00%> (-4.24%) |
:arrow_down: |
src/CoreBundle/Controller/UserController.php | 83.33% <0.00%> (-4.17%) |
:arrow_down: |
... and 458 more |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
@christianbeeznest no veo migración ninguna aquí. No se supone que haya una migración desde 1.11.* hacia master porque en 1.11.* también pueden haber usuarios suscritos a un LP?
Cierto @ywarnier , estaba pendiente esa parte, me asigne de nuevo la tarea para revisarlo, gracias
Cierto @ywarnier , estaba pendiente esa parte, me asigne de nuevo la tarea para revisarlo, gracias
Agregado archivos de migracion en el siguiente commit.
I close this PR due conflicts with last version