Skip to content

Commit

Permalink
fix PHP8.1 problem where Types::parseAsciiStringFromRegister() fail…
Browse files Browse the repository at this point in the history
…s 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
  • Loading branch information
aldas committed Dec 5, 2021
1 parent c39a0e5 commit 4418c74
Show file tree
Hide file tree
Showing 8 changed files with 640 additions and 261 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/php.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ jobs:
strategy:
matrix:
operating-system: [ubuntu-latest]
php-version: ['7.3', '7.4', '8.0']
php-version: ['7.4', '8.0', '8.1']
name: PHP ${{ matrix.php-version }} Test on ${{ matrix.operating-system }}
steps:
- uses: actions/checkout@v2
Expand All @@ -31,8 +31,8 @@ jobs:
run: composer test-ci

- name: Upload coverage to Codecov
if: matrix.php-version == '7.4' && matrix.operating-system == 'ubuntu-latest'
uses: codecov/codecov-action@v1
if: matrix.php-version == '8.1' && matrix.operating-system == 'ubuntu-latest'
uses: codecov/codecov-action@v2
with:
token: ${{ secrets.CODECOV_TOKEN }}
file: ./report/coverage.xml
Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -2,3 +2,4 @@

/vendor/
/report/
composer.phar
12 changes: 12 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,18 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [2.3.1] - 2021-12-05

### Added

* Use PHP 8.1 in CI flow.
* `examples/index.php` now supports FC3 and FC4 requests.

### Fixed

* From PHP 8.1 `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.

## [2.3.0] - 2021-05-09

### Added
Expand Down
3 changes: 2 additions & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@
"type": "library",
"license": "Apache-2.0",
"require": {
"php": "^7.0 || ^8.0"
"php": "^7.0 || ^8.0",
"ext-mbstring": "*"
},
"require-dev": {
"phpunit/phpunit": "^9.0",
Expand Down
Loading

0 comments on commit 4418c74

Please sign in to comment.