Pwirth has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/323813

Change subject: BSReview: Fixed accidentally return when good Status
......................................................................

BSReview: Fixed accidentally return when good Status

=> NEEDS merge to REL1_27 and master

Change-Id: Ia64aaa3168626d3adf02b7a540ece0cd11448d97
---
M Review/Review.class.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/BlueSpiceExtensions 
refs/changes/13/323813/1

diff --git a/Review/Review.class.php b/Review/Review.class.php
index ac07e2e..eb16090 100644
--- a/Review/Review.class.php
+++ b/Review/Review.class.php
@@ -1361,7 +1361,7 @@
                        $oStatus,
                ));
 
-               if( $oStatus->isOK() ) {
+               if( !$oStatus->isOK() ) {
                        return $oStatus->getMessage();
                }
 

-- 
To view, visit https://gerrit.wikimedia.org/r/323813
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia64aaa3168626d3adf02b7a540ece0cd11448d97
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/BlueSpiceExtensions
Gerrit-Branch: REL1_23
Gerrit-Owner: Pwirth <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to