Skip to content

Commit

Permalink
[phalcon#16593] - fix: fixed memory leak by anonymous function in Pas…
Browse files Browse the repository at this point in the history
…calCase.zep
  • Loading branch information
noone-silent committed May 24, 2024
1 parent 041897a commit 32630bd
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 8 deletions.
12 changes: 12 additions & 0 deletions CHANGELOG-5.0.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,17 @@
# Changelog

## [5.7.1](https://github.com/phalcon/cphalcon/releases/tag/v5.7.1) (XXXX-XX-XX)

### Changed

### Added

### Fixed

- Fixed `Phalcon\Support\Helper\PascalCase` causing memory leak by anonymous function [#16593](https://github.com/phalcon/cphalcon/issues/16593)

### Removed

## [5.7.0](https://github.com/phalcon/cphalcon/releases/tag/v5.7.0) (2024-05-17)

### Changed
Expand Down
14 changes: 6 additions & 8 deletions phalcon/Support/Helper/Str/PascalCase.zep
Original file line number Diff line number Diff line change
Expand Up @@ -25,18 +25,16 @@ class PascalCase
string text,
string delimiters = null
) -> string {
var exploded, output;
var exploded, output, element;

let exploded = this->processArray(text, delimiters);

let output = array_map(
function (element) {
return ucfirst(mb_strtolower(element));
},
exploded
);
let output = "";
for element in exploded {
let output = output . ucfirst(mb_strtolower(element));
}

return implode("", output);
return output;
}

/**
Expand Down

0 comments on commit 32630bd

Please sign in to comment.