<div dir="ltr"><div>I was just going to fix it as part of my test pull request.  </div><div><br></div><div>It really does not matter to me what the next steps are.  It is pretty trivial.   I you think I need practice making a PR, and then fixing it with a pull request.  Then we can do that as well.  </div><div><br></div><div>I also have no problem with you just fixing it.   </div><div><br></div><div>Stefan</div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Sun, Jan 25, 2026 at 12:08 PM John Ralls <<a href="mailto:jralls@ceridwen.us">jralls@ceridwen.us</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Stefan,<br>
<br>
Good catch. Obviously a copy-paste oversight and nearly 20 years old. I don’t see any access to idata via the property, only through the explicit setter and getter, so no real impact. That would explain why nobody noticed.<br>
<br>
Would you like to submit a PR or shall I fix it?<br>
<br>
Regards,<br>
John Ralls<br>
<br>
> On Jan 25, 2026, at 08:46, Stefan Koch <<a href="mailto:stefan.koch.micro@gmail.com" target="_blank">stefan.koch.micro@gmail.com</a>> wrote:<br>
> <br>
> While trying to learn some of the systems in the source code through writing unit tests, I think I found a bug in the way the qofinstance properties are setup.  (I don't see much usage of this so it may not be an issue in practice.)  <br>
> <br>
> I think the following change fixes the issue:  <br>
> <br>
> modified   libgnucash/engine/qofinstance.cpp<br>
> @@ -244,7 +244,7 @@ static void qof_instance_class_init(QofInstanceClass *klass)<br>
>  <br>
>      g_object_class_install_property<br>
>      (object_class,<br>
> -     PROP_EDITLEVEL,<br>
> +     PROP_IDATA,<br>
>       g_param_spec_uint ("idata",<br>
>                          "Object IData",<br>
>                          "Per instance backend private data.",<br>
> <br>
> Since I am new, I would like to confirm before going on.  <br>
> <br>
> Stefan<br>
> _______________________________________________<br>
> gnucash-devel mailing list<br>
> <a href="mailto:gnucash-devel@gnucash.org" target="_blank">gnucash-devel@gnucash.org</a><br>
> <a href="https://lists.gnucash.org/mailman/listinfo/gnucash-devel" rel="noreferrer" target="_blank">https://lists.gnucash.org/mailman/listinfo/gnucash-devel</a><br>
<br>
</blockquote></div>