Notice when using comma to separate orderby columns

tessus / mwExtensionMantis

MediaWiki Extension:Mantis

https://www.mediawiki.org/wiki/Extension:Mantis

GNU General Public License v2.0

2 stars 6 forks source link

Notice when using comma to separate orderby columns #33

Closed pedwik closed 7 years ago

pedwik commented 7 years ago

The following example raises a warning when using the orderby argument with two columns separeate by a comma. Message is "Notice: Undefined offset ...Mantis.php on line 329"

status = resolved, closed fixed_in_version = 2.8.1, 2.9.0 show = id,category, summary, fixed_in_version, updated, resolution orderby = category, summary

tessus commented 7 years ago

Will have a look at it later today.

tessus commented 7 years ago

What is the entire error message? (The ... part.)

tessus commented 7 years ago

Never mind. This also is only a notice and does not change the behaviour of the code.

The code still works.

tessus commented 7 years ago

fixed in https://github.com/tessus/mwExtensionMantis/commit/e27a428b74dcbf58f84b7cea97329b80599243a3

pedwik commented 7 years ago

The ... part is the file path to mantis.php

Mvh Peder

11 nov. 2017 kl. 01:22 skrev Helmut K. C. Tessarek notifications@github.com:

What is the entire error message? (The ... part.)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.