I don’t think it’s needed to cancel the PR, now that it’s there already. A PR to be open and actively improved is totally normal. Just keep the process clear by commenting from time to time.
Regarding the api load problem: Synchronizing requests sounds like the right approach. You could check other bindings how something like that is done there. Otherwise you could contact developers of similar bindings. A less technical solution would be to simply warn users in the README to not overuse their API key, considering their things count and the configured interval. That would be an okay solution for a first version.
thanks to @opus, who supported me a lot with his experience with the tankerkoenig api, I changed a bit the binding handling in order to save some superfluous api requests.
Additionally I added the README.md with a little description how to set up the binding.
Besides the need to restart before any data is fetched from tankerkonig.de I have observed no errors over the day of testing ( using 3 fuel-stations).
Thanks for this binding.
Did check how often the prices were fetched, I can’t see the 10 minutes I did set at all!
I do see steps between 1 and 10 minutes!
I’ll stop the binding for now.
I have made a “sidestep” and created a rule that does what I intend to do with the binding regarding opening times. That is working! Posted here
Now I’m trying to get the same working in Eclipse with the binding. I do have problems using either the feature the author used (make another polling job, use the *service to get data). But I’ll get there, but it will take some more time.
It seems to me that I’m at a dead end street in trying to adjust the binding using Eclipse.
Could anybody try to help? Or should I ask on github?
I used the code build by @dolic and added additional code which mimics the already used classes and methods “TankerkoenigListXXX” and added “TankerkoenigDetailXXX” (including all needed sub-types). The difference between both is that the later uses the “detail.php” to get detail data.
For using them I also created a “TankerkoenigDetailService” with the “GetTankerkoenigDetailResult” method.
When trying to call it inside a response job using this code:
logger.info("UpdateTankstellenDetails jsonData : {}", jsonData);
GsonBuilder gsonBuilder = new GsonBuilder();
gsonBuilder.registerTypeAdapter(TankerkoenigDetailResult.class,
new CustomTankerkoenigDetailResultDeserializer());
Gson gson = gsonBuilder.create();
logger.info("UpdateTankstellenDetails before creating res");
TankerkoenigDetailResult res = gson.fromJson(jsonData, TankerkoenigDetailResult.class);
logger.info("UpdateTankstellenDetails res : {}", res);
I get correct log-entries for the jsonData (I verfied the reported string) and as a last entry “UpdateTankstellenDetails before creating res”. After that I get no more entries.
[Edit:]
The problem is my " CustomTankerkoenigDetailResultDeserializer", but I’m unable to solve that one.
i dont get the binding running. The itemvalues just show NULL an i got this in my logs:
18:09:47.696 [ERROR] [ome.core.thing.internal.ThingManager] - Exception occurred while initializing handler of thing 'tankerkoenig:config:a174bea6': java.lang.NullPointerException
java.util.concurrent.ExecutionException: java.lang.NullPointerException
at java.util.concurrent.FutureTask.report(FutureTask.java:122)[:1.8.0_131]
at java.util.concurrent.FutureTask.get(FutureTask.java:206)[:1.8.0_131]
at org.eclipse.smarthome.core.common.SafeMethodCaller.callAsynchronous(SafeMethodCaller.java:194)[99:org.eclipse.smarthome.core:0.9.0.201705120951]
at org.eclipse.smarthome.core.common.SafeMethodCaller.call(SafeMethodCaller.java:83)[99:org.eclipse.smarthome.core:0.9.0.201705120951]
at org.eclipse.smarthome.core.common.SafeMethodCaller.call(SafeMethodCaller.java:67)[99:org.eclipse.smarthome.core:0.9.0.201705120951]
at org.eclipse.smarthome.core.thing.internal.ThingManager$9.run(ThingManager.java:710)[106:org.eclipse.smarthome.core.thing:0.9.0.201705120951]
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)[:1.8.0_131]
at java.util.concurrent.FutureTask.run(FutureTask.java:266)[:1.8.0_131]
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:180)[:1.8.0_131]
at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:293)[:1.8.0_131]
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)[:1.8.0_131]
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)[:1.8.0_131]
at java.lang.Thread.run(Thread.java:748)[:1.8.0_131]
Caused by: java.lang.NullPointerException
at org.openhab.binding.tankerkoenig.handler.BridgeHandler.initialize(BridgeHandler.java:82)[237:org.openhab.binding.tankerkoenig:2.1.0.201706012056]
at org.eclipse.smarthome.core.thing.internal.ThingManager$9$1.call(ThingManager.java:713)[106:org.eclipse.smarthome.core.thing:0.9.0.201705120951]
at org.eclipse.smarthome.core.thing.internal.ThingManager$9$1.call(ThingManager.java:1)[106:org.eclipse.smarthome.core.thing:0.9.0.201705120951]
at org.eclipse.smarthome.core.common.SafeMethodCaller$CallableWrapper.call(SafeMethodCaller.java:181)[99:org.eclipse.smarthome.core:0.9.0.201705120951]
at java.util.concurrent.FutureTask.run(FutureTask.java:266)[:1.8.0_131]
... 3 more
Could you give some details on your setup?
Where did you get the binding from? Since the pull request isn’t approved yet, the most actual.jar can be downloaded on the eclipse marketplace (last one from yesterday)
How did you setup your system? Did you create a bridge with your personal Api key?
Did you create any 'Tankstelle" with the respective ID?
Are you using openHAB2? This binding is made for that version! Link to the Marketplace
i installed it with the paperui… there it appears as a marketplace-binding, i am using openhab2, snapshot of last week.
on the homepage i created a api-key, then i added the configuration as a thing with the api-key, then i added three tankstellen as things with their ids. of course i choose the configuration-thing as bridge (was the only value i could choose).
then i created a linked item for one of the tankstellen.
all the created things have the status “UNINITIALIZED - HANDLER_INITIALIZING_ERROR” in the paperui things overview
It looks like you are doing everything correct!
You are on a snapshot release which I’m not, however that should not explain that error.
Can you give me your exact binding version ( from the karaf console use bundle:list). That way I could pinpoint where the error is coming from, or if it might be resolved already (don’t think so)
I not at home, so actual help by changing code does have to wait until the end of the weekend, I’m sorry.
Thanks, although that was stupid question of me, since that is reported in the error you posted already.
You are using the version of last night, so the is nothing newer!
I’m not sure ,but thisLink the my Dropbox might give you on older version, that was reported working.
Will look into the one on the marketplace when I ’ m back.
That should work under Configuration -Things, clicking on the plus sign should show the dialog from which binding to add things…
Let me know if that doesn’t work either.
In this case you have to wait until I’m back home. I already have an idea what causes the error, but I need to check.
I’ll write in here if I have something new. Is that OK with you?
