You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@sling.apache.org by GitBox <gi...@apache.org> on 2021/11/24 12:20:39 UTC

[GitHub] [sling-org-apache-sling-testing-osgi-mock] stefanseifert commented on pull request #14: SLING-10930 add failing test with multiple deactivate() calls on the

stefanseifert commented on pull request #14:
URL: https://github.com/apache/sling-org-apache-sling-testing-osgi-mock/pull/14#issuecomment-977825162


   i've implemented a simple fix as proposal for this issue, to ensure we are not calling deactivate multiple times.
   
   sidenote: i've seen that the getService() method used in this fix is a) ill-named and b) will lead to problems when a ServiceFactory is used (something that is probably used very rarely and i think not correctly supported currently anyways - this will be subject to another issue)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@sling.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org