Skip to content

[Analysis] Statement.php: mixed-assignment in bindParametersFromContainer() stems from ParameterContainer::getNamedArray() #67

Closed

Description

@tyrsson

Context

Part of #60.

Data

mago analyze flags mixed-assignment on $value in bindParametersFromContainer()'s foreach ($parameters as $name => &$value) loop. $parameters comes from ParameterContainer::getNamedArray(): array (upstream php-db/phpdb, docblock @return array<string|int, mixed>). This is not an untyped-boundary bug: bind parameter values are legitimately heterogeneous scalars (int/string/float/null) by design, so mixed is an accurate description of the value, not a gap to close.

These findings will be captured in a mago baseline rather than fixed or suppressed inline.

Activity

  1. added this to the 0.5.0 milestone on Aug 10, 2026
  2. added theissue type on Aug 10, 2026
  3. tyrsson commented on Oct 7, 2026

    @tyrsson
    MemberAuthor

    Closing as resolved. The mixed-assignment on \ in bindParametersFromContainer() is captured in the committed analysis-baseline.toml for src/Statement.php, per the disposition recorded in this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions