You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Throughout the code base there are some calls to utf8::upgrade(). These are perfectly fine as we want Perl strings with internal UTF-8 encoding. However most of these upgrade strings that come from Kernel::System::YAML::Dump(). That method already upgrades the generated YAML strings. So I propose to remove the redundant calls. Especially as I fear that utf8::upgrade() is not employed consistently. The utf8::upgrade()in Kernel::System::YAML must of course stay intact.
Throughout the code base there are some calls to
utf8::upgrade()
. These are perfectly fine as we want Perl strings with internal UTF-8 encoding. However most of these upgrade strings that come fromKernel::System::YAML::Dump()
. That method already upgrades the generated YAML strings. So I propose to remove the redundant calls. Especially as I fear thatutf8::upgrade()
is not employed consistently. Theutf8::upgrade()
in Kernel::System::YAML must of course stay intact.The text was updated successfully, but these errors were encountered: