jenkins-bot has submitted this change and it was merged. Change subject: Return true from job&execute() ......................................................................
Return true from job&execute() The listener script expects it Change-Id: I2d4630370b752b6d9c7b375dd5ddad543fc4275b --- M PaymentProviders/PayPal/Job.php 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Awight: Looks good to me, approved jenkins-bot: Verified diff --git a/PaymentProviders/PayPal/Job.php b/PaymentProviders/PayPal/Job.php index c1b7711..00848e0 100644 --- a/PaymentProviders/PayPal/Job.php +++ b/PaymentProviders/PayPal/Job.php @@ -68,5 +68,8 @@ $this->config->object( 'data-store/' . $msg_type ) ->push( $new_msg ); + // TODO It would be nice if push() returned something useful so we + // could return something here too + return true; } } -- To view, visit https://gerrit.wikimedia.org/r/311910 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d4630370b752b6d9c7b375dd5ddad543fc4275b Gerrit-PatchSet: 1 Gerrit-Project: wikimedia/fundraising/SmashPig Gerrit-Branch: master Gerrit-Owner: Cdentinger <cdentin...@wikimedia.org> Gerrit-Reviewer: Awight <awi...@wikimedia.org> Gerrit-Reviewer: Ejegg <eeggles...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits