phalcon / phalcon/cphalcon

[BUG]: Volt: dead closure prefix

Open
#17,564 0 comments 0 reactions 1 assignee View on GitHub

@niden is already working on this.

Since Sep 4, 2026.

7.0 bug status: medium
Dominant language
PHP
Stars
10.8k
Forks
1.9k
Avg merge
7h 2m
Merged PRs (30d)
47

Description

The closure prefix generator cannot be used

Compiler::getUniquePrefix() has a branch that calls the prefix as a closure, but setUniquePrefix() takes a string in both projects, so the only way in is a subclass writing the protected property. cphalcon declares protected prefix (untyped) so that works; phalcon has since widened its property to match, but the setter still refuses a closure on both sides.

Decide whether the closure prefix is supported. If yes, widen setUniquePrefix() in both. If no, drop the branch from both.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.