vimeo/psalm

`str_replace()` with more search then replace values

Open

#4,253 建立於 2020年9月28日

在 GitHub 查看
 (4 留言) (0 反應) (0 負責人)PHP (668 fork)batch import
Help wantedenhancementtype providers

倉庫指標

Star
 (5,369 star)
PR 合併指標
 (平均合併 3天 12小時) (30 天內合併 5 個 PR)

描述

even if it is supported by php-src, I think it should be an error (or at least warning) when using str_replace with less replace args, then search args. its most likely a typo and the default behaviour of php-src assuming we want to replace with an empty string is not obvious.

I expect an error, telling me that the number of replace args is not equal with the number of search args in https://psalm.dev/r/7385fe7881

貢獻者指南

研究方向
研究 Psalm 如何檢測函數調用中的問題。查看類似內建函數（例如陣列函數）的現有實現，並為 str replace 建立一個新的檢查，當搜尋參數數量超過替換參數數量時發出警告。
技術棧
php
領域
devtools
議題類型
功能
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
3
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-2 天
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
PHPGit
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
65