gajus / gajus/moa

Setter is assuming unicode encoding

Open
#2 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
PHP
Stars
49
Forks
2
PR merge metrics
No merged PRs in 30d

Description

MySQL encoding does not seem to translate directly to PHP character encoding names. Setter is assuming that value is unicode and might produce unexpected results.

```
if (!is_null(static::$columns[$name]['character_maximum_length']) && static::$columns[$name]['character_maximum_length'] < mb_strlen($value)) {
throw new Exception\InvalidArgumentException('Property does not conform to the column\'s maxiumum character length limit.');
}
```

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.