2amigos / yiiwheels

Widget extension library for the YiiStrap extension
http://yiiwheels.2amigos.us
Other
133 stars 77 forks source link

Call to undefined method WhHtml::getOption() #35

Closed addicted2sounds closed 11 years ago

addicted2sounds commented 11 years ago

I get this error trying to use WhDateRangePicker as it's mentioned in the docs. Haven't tried other widgets using this function

tonydspaniard commented 11 years ago

getOption is a function coming from its parent class TbHtml. Removed WhHtml support, I fix it asap

tonydspaniard commented 11 years ago

Strange.. https://github.com/2amigos/yiiwheels/blob/master/widgets/daterangepicker/WhDateRangePicker.php does not have support for WhHtml anymore.. how did you get this error?

tonydspaniard commented 11 years ago

@addicted2sounds Apologies... but it seems Crisu modified Yiistrap library and removed support for TbHtml::getOption. I have to modify all widgets to use TbArray instead when I have time. Or, instead of using Cris, use the following: https://github.com/tonydspaniard/yiistrap

Which still has support for TbHtml::getOption

tonydspaniard commented 11 years ago

Works also with Yiistrap 1.0.0-beta version https://github.com/Crisu83/yiistrap/tree/1.0.0-beta

I try to fix it asap on the new version (no much time now)

addicted2sounds commented 11 years ago

I used you intermediate YiiInitializer template. I'm a little confused about what packages should I use to make yiistrap and yiiwheels working. 2amigos/yiiwheels mentioned in template's composer.json requires 2amigos/yiistrap that as I understand is not supported (also it has an error in TbHtml), because official website for yiistrap shows crisu83 repository. Could you let me know if I still can use composer any way to update application?

tonydspaniard commented 11 years ago

@addicted2sounds I will fix everything so there wont be any more issues regarding yiistrap - yiiwheels. You won't be any longer confused.

2amigos/yiistrap is our own version of yiistrap repo as there were continuous changes on the library and we were already using them in our projects.

I will make a new rebuild and make sure there are no issues related to the libraries anymore. Also, from Yiinitializr.

addicted2sounds commented 11 years ago

Thanks, for your support and great extensions

tonydspaniard commented 11 years ago

@addicted2sounds the issue was solved....

YiiWheels 1.0.2 is up to date with latest changes on YiiStrap 1.0.1 (version maintained by us of YiiStrap Chris - this one has the latest bug fixes and new additions of RC1)

YiiWheels 1.0.1 works with previous YiiStrap 1.0.0 (In order not to break BC)

Hope these changes help you.

Cheers!