Skip to content

Pass an object rather than a handle to the note editor callback #2070

New issue

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

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

Already on GitHub? Sign in to your account

Merged

Conversation

stevenyoungs
Copy link
Contributor

Fix EditNote so that it passes the object when calling the callback function, rather than the object handle.
Fixes the error described in 13702 and gives consistency with other Edit classes.

To reproduce:

  1. create a new note- type "a"
  2. select "a" and click the Link button
  3. Change the Link Type to Note and click New
  4. type "a" and click ok

Fixes #13702 and possibly #12260

Reviewed and adjusted other places that create an EditNote instance with callback

@hgohel
Copy link
Member

hgohel commented May 22, 2025

Looks good to me!

@hgohel hgohel added the bug label May 24, 2025
@dsblank dsblank added this to the v6.0 milestone Jun 7, 2025
@Nick-Hall Nick-Hall changed the title Bug 13702: EditNote Callback: Pass object not handle EditNote Callback: Pass object not handle Jun 9, 2025
@Nick-Hall Nick-Hall changed the title EditNote Callback: Pass object not handle Pass an object rather than a handle to the note editor callback Jun 10, 2025
This gives consistency with other Edit classes.
Adjust other code accordingly.

Fixes #13702.
@Nick-Hall Nick-Hall merged commit 1b68f87 into gramps-project:maintenance/gramps60 Jun 10, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants