• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

KarpelesLab / goro / 26351382778
12%

Build:
DEFAULT BRANCH: master
Ran 24 May 2026 04:02AM UTC
Jobs 1
Files 398
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

24 May 2026 04:01AM UTC coverage: 12.473% (+0.01%) from 12.46%
26351382778

push

github

MagicalTux
vm: unwrap receiver + caller-scope private resolution in CallInstanceMethodByExprs

CallInstanceMethodByExprs (the B9-Stage1 helper that handles object
method calls with by-ref/named/spread args) was missing two pieces of
the AST runner's dispatch logic that the existing CallInstanceMethod
path also lacks but happens to not be exercised by tests:

1. Unwrap the receiver's kin marker before method lookup so a parent
   method calling $this->childMethod() resolves through the actual
   runtime class's vtable, not the narrowed parent class. Without
   this, add_optional_by_ref_arg.phpt regressed (Test1.method1 called
   Test1.method2 instead of Test2.method2 on a Test2 instance).

2. Re-resolve private methods through the caller's class scope.
   Private methods are not virtual: when calling $this->method()
   from within a class that defines a private method with that name,
   PHP uses the caller's private method regardless of what the
   runtime class hierarchy provides. Without this, bug69180-backtrace
   regressed (P extends Model, both define private __t_get; calling
   $this->__t_get from Model.__get on a P instance found P's private
   method and threw a visibility error).

Both fixes mirror runObjectFunc.Run (compile-object.go ~880,~912).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

13 of 21 new or added lines in 1 file covered. (61.9%)

14742 of 118194 relevant lines covered (12.47%)

2579.01 hits per line

Uncovered Changes

Lines Coverage ∆ File
8
47.15
0.43% core/compiler/dispatch.go
Jobs
ID Job ID Ran Files Coverage
1 26351382778.1 24 May 2026 04:02AM UTC 398
12.47
GitHub Action Run
Source Files on build 26351382778
  • Tree
  • List 398
  • Changed 2
  • Source Changed 1
  • Coverage Changed 2
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #26351382778
  • 511c1b08 on github
  • Prev Build on master (#26351198757)
  • Next Build on master (#26351462893)
  • Delete
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc