⚠ In case you've missed it, we have migrated to our new website, with a brand new forum. For more details about the migration you can read our blog post for website migration. This is an archived forum. ⚠

  •     

profile picture

possible problem with 1.2.1.1



jcasanova

jcasanova
  • profile picture
  • Member

Posted 24 April 2012 - 18:04 PM

In the library there's this lines:


protected $default_javascript_path = 'assets/grocery_crud/js';
protected $default_css_path = 'assets/grocery_crud/css';
protected $default_texteditor_path = 'assets/grocery_crud/texteditor';
protected $default_theme_path = 'assets/grocery_crud/themes';
protected $default_language_path = 'assets/grocery_crud/languages';


I moved the assets dir and doesn't work.... is that for those vars are?

because in list_template.php are this others:


$this->set_css('assets/grocery_crud/themes/flexigrid/css/flexigrid.css');
$this->set_js('assets/grocery_crud/js/jquery-1.7.1.min.js');
$this->set_js('assets/grocery_crud/themes/flexigrid/js/cookies.js');
$this->set_js('assets/grocery_crud/themes/flexigrid/js/flexigrid.js');
$this->set_js('assets/grocery_crud/themes/flexigrid/js/jquery.form.js');
$this->set_js('assets/grocery_crud/themes/flexigrid/js/jquery.numeric.js');

[b]this should be...[/b]

[b]$this->set_css($this->default_theme_path.'/flexigrid/css/flexigrid.css');[/b]
[b]$this->set_js($this->default_javascript_path.'/jquery-1.7.1.min.js');[/b]
[b]$this->set_js($this->default_theme_path.'/flexigrid/js/cookies.js');[/b]
[b]$this->set_js($this->default_theme_path.'/flexigrid/js/flexigrid.js');[/b]
[b]$this->set_js($this->default_theme_path.'/flexigrid/js/jquery.form.js');[/b]
[b]$this->set_js($this->default_theme_path.'/flexigrid/js/jquery.numeric.js');[/b]

that way it works.

@web-johnny is it a bug?

web-johnny

web-johnny
  • profile picture
  • Administrator
  • 1,166 posts

Posted 24 April 2012 - 20:29 PM

Yes this is right. I just didn't have the time to finished that, that's why I didn't even announce it. Probably I will fix that at version 1.2.2 but not sure.
I am trying to move all the hardcoded javascripts and css to one path that the user will change it from the config. I just didn't finished it yet.

Thank you to reminding me this.

P.S. Your fix is right just make sure that all the files are like this.

jcasanova

jcasanova
  • profile picture
  • Member

Posted 26 April 2012 - 13:42 PM

Oh ok, I think that could be the reason =).

Nice work!

Matt

Matt
  • profile picture
  • Member

Posted 09 July 2012 - 11:47 AM

I would like to do this as well. I'd rather not hack the core, is this still on the roadmap?

web-johnny

web-johnny
  • profile picture
  • Administrator
  • 1,166 posts

Posted 09 July 2012 - 19:02 PM

Nope this issue is fixed as for version 1.2.2