Nauktis / SolarFlux

Mod for Minecraft - Adds solar panels that produce Redstone Flux
GNU General Public License v3.0
15 stars 8 forks source link

New cofh api #12

Closed grydian closed 9 years ago

grydian commented 9 years ago

Since the new cofh api I cant get the solars to power EnderIO machines directly anymore. I have have to use a cable to connect the solars to the farming station. Be the updates to cofhcore and enderio it was working perfectly. Just wanted to let you know.. Thanks for a great mod.

Nauktis commented 9 years ago

Will investigate! Thanks for reporting the issue.

Nauktis commented 9 years ago

New version of the API deployed. Please come back to me if the bug is not resolved. Cheers,

grydian commented 9 years ago

Its still not powering the EnderIO farming station like it used to :( Just tested with the latest version of both mods and latest cofhcore and thermal expansion,foundation

Nauktis commented 9 years ago

I have it working in ModSauce (might not be the latest version of EnderIO / COFH mods). Are you using version SolarFlux-1.7.10-0.4d.jar of my mod?

It seems EnderIO is including the cofh apis in its Jar file which is a bad practice, I opened an issue on EnderIO's side. My mod used to do it as well.

Does the solar panels power Thermal Expansion machines at least?

grydian commented 9 years ago

Yes I am using version d. I will do more testing with TE machines. I can also edit the rar of enderio and take out the api and see if that helps to narrow it down for you. I will also compare my pack with modsauce and see whats different. Thanks man.

grydian commented 9 years ago

Just check the mod sauce pack and yes its using the old version of cofhcore and thats the version that everything worked fine. The new version is not working however. The version I am using is CoFHCore-[1.7.10]3.0.0B9-40. I am however using that version of EnderIO 311. Further testing: The solars will power energy cells and capacitor banks but not TE machines or EnderIO machines directly anymore without a cable in between. If you need any more information let me know.

grydian commented 9 years ago

Just to let you know I tested again with the latest solarflux .5a and I also updated cofh core to b9-71. That was the absolute latest I could find. I also updated all the teamcofh mods to the latest betas I could find and I still have to use a conduit to power machines directly with the solar panels. Thanks for all your help man.

Barhandar commented 9 years ago

@Nauktis You're checking for the IEnergyHandler, which isn't used by machines anymore, only various energy cells. You need to check for IEnergyReceiver instead to make it work with major mods. It won't break compatibility with energy cells and mods like GC and Mekanism since "IEnergyHandler" is technically just a shorthand for "IEnergyReceiver, IEnergyProvider".

P.S. SES needs the aforementioned change too.

Nauktis commented 9 years ago

@Barhandar Damn I should have seen your message earlier. Thanks for reporting man. I just pushed a fix ;)