Santhosh has uploaded a new change for review.

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

Change subject: Fix PHP error in ApiContentTranslationDelete
......................................................................

Fix PHP error in ApiContentTranslationDelete

The $translation->update() call was failing because Translation.php
required a mandatory options param for update method. Made it optional

Bug: T129213
Change-Id: I5c9bfc42d73efb44181188584873e699c702ef6e
---
M includes/Translation.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/ContentTranslation 
refs/changes/08/276708/1

diff --git a/includes/Translation.php b/includes/Translation.php
index ba5ee55..fb01dcc 100644
--- a/includes/Translation.php
+++ b/includes/Translation.php
@@ -42,7 +42,7 @@
                $this->translation['id'] = (int)$dbw->insertId();
        }
 
-       public function update( array $options ) {
+       public function update( array $options = null ) {
                $dbw = Database::getConnection( DB_MASTER );
 
                $values = array(

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5c9bfc42d73efb44181188584873e699c702ef6e
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ContentTranslation
Gerrit-Branch: master
Gerrit-Owner: Santhosh <santhosh.thottin...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to