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

[husky2]: bug fix #930

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

Conversation

gruberth
Copy link
Contributor

@gruberth gruberth commented Apr 9, 2024

Fixed a endless loop wich was caused by a wrong/outdated check if the update_item() call came from the plugin itself.

@gruberth gruberth changed the title husky2bug fix [husky2]: bug fix Apr 9, 2024
@msinn
Copy link
Member

msinn commented May 5, 2024

I don't get, which problem is fixed by this PR.

  • get_fullname() is only needed in multiinstance capable plugins to distinguish the instances of the plugin. It adds the instance name to the plugin nam. This plugin is not mulitinstance capable.

  • the caller parameter is a string. The comparision caller != self.get_shortname() (or caller != self.get_fullname() ). Why are you looking for a substring?

@gruberth
Copy link
Contributor Author

gruberth commented May 5, 2024

Nach dem erfolgreichen Senden von Befehlen werden die entsprechedenen (Bool-) Items vom Plugin selbst wieder auf False zurückgesetzt (caller ist dabei auf fullname gesetzt). Da aber der caller in der update_item() manchmal als Eval:husky2 anstatt nur husky2 ankommt, suche ich einfach nach dem Substring. Wenn ich das nicht mache, habe ich ab und zu eine Endlosschleife in der update_tiem() nachdem ich einen Befehl sende und sh verabschiedet sich.

Ob ich fullname oder shortname verwende ist hier logisch egal, aber ist auch nicht schlimm wenns der fullname ist.

PS: dieser Fehelr tritt nur im develop branch auf.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants