Skip to content
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

Some adjustments to make version 6 work #13

Merged
merged 2 commits into from
Jun 6, 2024
Merged

Conversation

cyppe
Copy link
Contributor

@cyppe cyppe commented Jun 6, 2024

The only critical thing found so far was:

getBinLogCurrent() method has changed into property binLogCurrent.

Worth noting to users of this library is that the format when listening to events like UpdateRowsDTO has changed.

Before I received values like this:

        $updatedRows    = $event->getValues();
        $tableName      = $event->getTableMap()->getTable();
        $eventTimestamp = $event->getEventInfo()->getTimestamp();

Now I had to to change in my application into:

        $updatedRows    = $event->values;
        $tableName      = $event->tableMap->table;
        $eventTimestamp = $event->getEventInfo()->timestamp;

For bigger applications and uses there is probably a lot more affected areas so v6 has breaking changes worth to mention in changelog when it's time to release it.

@huangdijia huangdijia merged commit 155c7e1 into huangdijia:main Jun 6, 2024
2 checks passed
# 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