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

Error in bat.lua when battery is 100% but still on A/C #209

Closed
foxtrot opened this issue Jul 6, 2016 · 3 comments
Closed

Error in bat.lua when battery is 100% but still on A/C #209

foxtrot opened this issue Jul 6, 2016 · 3 comments

Comments

@foxtrot
Copy link

foxtrot commented Jul 6, 2016

Seems when the battery is on 100% and still plugged in it does not change to say "A/C", but if I restart awesomewm it will say "A/C", but error out every time the applet is updated with "bat.lua:148: attempt to compare string with number"

@foxtrot
Copy link
Author

foxtrot commented Jul 6, 2016

I should mention I am using the latest lain from the AUR package.

lcpz pushed a commit that referenced this issue Jul 6, 2016
@lcpz
Copy link
Owner

lcpz commented Jul 6, 2016

Assuming this happens when you wait widget timeout (30s default) and then you restart wm, I can't manage to replicate the issue.

If your widget is able to execute until that line, it means that bat_now.status == "Discharging" is true, and it shouldn't. A problem with buffers/ram?

Let's add a type checking. Try latest commit.

lcpz pushed a commit that referenced this issue Jul 6, 2016
@foxtrot
Copy link
Author

foxtrot commented Jul 6, 2016

Works great with latest commit!
Thank you very much.

@foxtrot foxtrot closed this as completed Jul 6, 2016
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

No branches or pull requests

2 participants