ApiMain.php on line 598
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Nikerabbit
	Apr 3 2012, 4:05 PM

Description

PHP Notice: Undefined index: gettoken in /www/w/includes/api/ApiMain.php on line 598

Version: 1.20.x
Severity: normal

Related Changes in GitLab:

Title	Reference	Author	Source Branch	Dest Branch
Sync up DagProperties of dumps_merge_events_to_wikitext_raw_daily with overrides.	repos/data-engineering/airflow-dags!906	xcollazo	update-dag-properties	main
Go back to copy-on-write for MERGE INTOs and DELETEs.	repos/data-engineering/mediawiki-content-pipelines!43	xcollazo	go-back-to-copy-on-write	main
builds-builder: add script to migrate pipelineruns from v1beta1 to v1	repos/cloud/toolforge/toolforge-deploy!554	dcaro	add_pr_v1_migration_script	main
d/changelog: bump to 1.6.4	repos/cloud/toolforge/toolforge-weld!61	dcaro	bump_toolforge-weld	main
d/changelog: bump to 1.6.4	repos/cloud/toolforge/toolforge-weld!60	dcaro	bump_version	main
api_client: increase timeout to 20s	repos/cloud/toolforge/toolforge-weld!59	dcaro	increase_timeout	main

		Status	Subtype	Assigned	Task
		Resolved		None	T37676 MediaWiki 1.17.4/1.18.3 maintenance release
		Resolved		hashar	T37671 PHP Notice: Undefined index: gettoken in includes/api/ApiMain.php on line 598

• bzimport raised the priority of this task from to Unbreak Now!.Nov 22 2014, 12:19 AM

• bzimport set Reference to bz35671.

Yay to whoever added gettoken only on few modules

if ( $salt !== false && !$moduleParams['gettoken'] ) {

if ( $salt !== false && isset( $moduleParams['gettoken'] ) && !$moduleParams['gettoken'] ) {

I guess...

comment 1 would bring back bug 34212

(In reply to comment #2)

comment 1 would bring back bug 34212

See commit message of https://gerrit.wikimedia.org/r/#change,4159

We're still doing the check on the value, but only if we have that parameter set, and hence something to check the value of...

I just tested it, that fix works fine, and doesn't regress on bug 34212