Skip to content

Commit 4418c74

Browse files
committed
fix PHP8.1 problem where Types::parseAsciiStringFromRegister() fails to convert extended ASCII (8bit) characters to UTF-8. Introduced Charset::$defaultCharset to be able to set default charset used to convert strings.
Use PHP8.1 in CI flow Add FC3 and FC4 selection to examples/index.php
1 parent c39a0e5 commit 4418c74

File tree

8 files changed

+640
-261
lines changed

8 files changed

+640
-261
lines changed

.github/workflows/php.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ jobs:
88
strategy:
99
matrix:
1010
operating-system: [ubuntu-latest]
11-
php-version: ['7.3', '7.4', '8.0']
11+
php-version: ['7.4', '8.0', '8.1']
1212
name: PHP ${{ matrix.php-version }} Test on ${{ matrix.operating-system }}
1313
steps:
1414
- uses: actions/checkout@v2
@@ -31,8 +31,8 @@ jobs:
3131
run: composer test-ci
3232

3333
- name: Upload coverage to Codecov
34-
if: matrix.php-version == '7.4' && matrix.operating-system == 'ubuntu-latest'
35-
uses: codecov/codecov-action@v1
34+
if: matrix.php-version == '8.1' && matrix.operating-system == 'ubuntu-latest'
35+
uses: codecov/codecov-action@v2
3636
with:
3737
token: ${{ secrets.CODECOV_TOKEN }}
3838
file: ./report/coverage.xml

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,3 +2,4 @@
22

33
/vendor/
44
/report/
5+
composer.phar

CHANGELOG.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,18 @@ All notable changes to this project will be documented in this file.
44
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
55
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
66

7+
## [2.3.1] - 2021-12-05
8+
9+
### Added
10+
11+
* Use PHP 8.1 in CI flow.
12+
* `examples/index.php` now supports FC3 and FC4 requests.
13+
14+
### Fixed
15+
16+
* From PHP 8.1 `Types::parseAsciiStringFromRegister()` fails to convert extended ASCII (8bit) characters to UTF-8. Introduced
17+
`Charset::$defaultCharset` to be able to set default charset used to convert strings.
18+
719
## [2.3.0] - 2021-05-09
820

921
### Added

composer.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@
77
"type": "library",
88
"license": "Apache-2.0",
99
"require": {
10-
"php": "^7.0 || ^8.0"
10+
"php": "^7.0 || ^8.0",
11+
"ext-mbstring": "*"
1112
},
1213
"require-dev": {
1314
"phpunit/phpunit": "^9.0",

0 commit comments

Comments
 (0)