Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

修复一些可能存在的数组键名是 int 类型时的错误 #563

Merged
merged 7 commits into from
Jul 13, 2023

Conversation

Yurunsoft
Copy link
Member

No description provided.

@Yurunsoft Yurunsoft added bug Something isn't working 2.1 2.0 labels Jul 11, 2023
@Yurunsoft Yurunsoft changed the title 修复一些可能存在的数组键名是 int 时的错误 修复一些可能存在的数组键名是 int 类型时的错误 Jul 11, 2023
@codecov
Copy link

codecov bot commented Jul 11, 2023

Codecov Report

Patch coverage: 83.33% and project coverage change: -0.59 ⚠️

Comparison is base (484fdc6) 65.45% compared to head (5aeeba0) 64.86%.

Additional details and impacted files
@@             Coverage Diff              @@
##                2.1     #563      +/-   ##
============================================
- Coverage     65.45%   64.86%   -0.59%     
  Complexity    11168    11168              
============================================
  Files           888      888              
  Lines         30459    30463       +4     
============================================
- Hits          19936    19759     -177     
- Misses        10523    10704     +181     
Impacted Files Coverage Δ
src/Components/jwt/src/Bean/JWT.php 0.00% <0.00%> (ø)
...roadrunner/src/Http/Message/RoadRunnerResponse.php 0.00% <0.00%> (ø)
src/Components/shared-memory/src/Client.php 0.00% <0.00%> (ø)
src/Components/swoole/src/Cron/CronManager.php 45.04% <0.00%> (ø)
...ver/ConnectionContext/StoreHandler/MemoryTable.php 56.25% <0.00%> (ø)
.../Components/swoole/src/Util/MemoryTableManager.php 73.33% <0.00%> (ø)
src/Db/Query/Query.php 59.82% <0.00%> (ø)
src/Cache/Handler/Apcu.php 100.00% <100.00%> (ø)
src/Cache/Handler/File.php 88.52% <100.00%> (ø)
src/Cache/Handler/Memory.php 95.00% <100.00%> (ø)
... and 29 more

... and 19 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Yurunsoft Yurunsoft added the optimization Optimization label Jul 12, 2023
@Yurunsoft Yurunsoft merged commit 919c540 into 2.1 Jul 13, 2023
50 of 54 checks passed
@Yurunsoft Yurunsoft deleted the fix-cache-setMultiple branch July 13, 2023 00:32
Yurunsoft added a commit that referenced this pull request Jul 14, 2023
* 修复缓存 setMultiple() 方法传参数组键名是数字

* 修复

* 修复一些可能存在的数组键名是 int 时的错误

* 修复一些可能存在的数组键名是 int 时的错误

* 优化代码

* 优化代码

* 格式化代码
# Conflicts:
#	dev/bootstrap.php
#	src/Cache/Handler/Memory.php
#	src/Cache/Handler/Redis.php
#	src/Cache/Handler/RequestContext.php
#	src/Components/grpc/src/Proxy/Http/GrpcHttpClient.php
#	src/Components/grpc/src/Proxy/Http/GrpcHttpProxy.php
#	src/Components/swoole/src/Cron/CronManager.php
#	src/Db/Drivers/TPdoStatement.php
#	tests/phpunit.xml
NHZEX pushed a commit to NHZEX/imi that referenced this pull request Oct 18, 2023
* 修复缓存 setMultiple() 方法传参数组键名是数字

* 修复

* 修复一些可能存在的数组键名是 int 时的错误

* 修复一些可能存在的数组键名是 int 时的错误

* 优化代码

* 优化代码

* 格式化代码
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.0 2.1 bug Something isn't working optimization Optimization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant