# [PATCH] zend_parse_method_parameters_ex: don't ignore flags

- Thread: https://externals.io/message/46501
- Original archive: https://news-web.php.net/php.internals/46501
- Started by Hans-Peter Oeri on 2009-12-23T12:05:49+00:00
- Messages: 2
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #46501 — Hans-Peter Oeri — 2009-12-23T12:05:49+00:00

URL: https://externals.io/message/46501#46501

Hi!

As of now, zend_parse_method_parameters_ex does ignore the ex-flags if
called in a procedural context.

NB: Calling it in a procedural context is explicitely handled in the
code! It's not a false use of the function.

I suppose, the body was copied from zend_parse_method_parameters and the
"flags" correction got lost in one case.

HP

---

## #46517 — Stanislav Malyshev — 2009-12-24T00:15:17+00:00

URL: https://externals.io/message/46501#46517 · In reply to: #46501

Hi!

> As of now, zend_parse_method_parameters_ex does ignore the ex-flags if
> called in a procedural context.
>
> NB: Calling it in a procedural context is explicitely handled in the
> code! It's not a false use of the function.
>
> I suppose, the body was copied from zend_parse_method_parameters and the
> "flags" correction got lost in one case.

Looks like it. Funny thing is there seems to be no core in the main repo 
using this particular function. But I committed the fix anyway, as it 
seems to be a clear bug.
-- 
Stanislav Malyshev, Zend Software Architect
stas@zend.com   http://www.zend.com/
(408)253-8829   MSN: stas@zend.com