Skip to content

Fix for #8565 #8566

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix for #8565 #8566

wants to merge 1 commit into from

Conversation

aafemt
Copy link
Contributor

@aafemt aafemt commented May 14, 2025

So far this is a simplest quick fix to let Pavel Zotov test it.

@aafemt
Copy link
Contributor Author

aafemt commented May 15, 2025

It looks like this fix also solves #7611, so workaround in gbak can be removed.

@aafemt
Copy link
Contributor Author

aafemt commented May 16, 2025

I think it would be better to merge this PR now because it will make firebirdtest.com operational and do the rest of postfixes in separate PRs one per issue.

@asfernandes
Copy link
Member

You will break even more things than you already broke in #8082 ;)

create table tests (
    id integer,
    str varchar(10) character set utf8,
    char_len integer,
    octet_len integer
);

create procedure proc (
    id integer,
    str char(10) character set utf8
)
as
begin
    insert into tests (id, str, char_len, octet_len) values (:id, :str, char_length(:str), octet_length(:str));
end;

@aafemt aafemt marked this pull request as draft May 19, 2025 07:48
@aafemt
Copy link
Contributor Author

aafemt commented May 19, 2025

Reworked.

@aafemt aafemt marked this pull request as ready for review May 19, 2025 14:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants