Typography "all_styles" parameter doesn't import all (Google) font weights

[aonzzung]

Hi there,

First, thanks for amazing theme option framework.
It's the best I have ever used. ;)

I'm having problem with "all_styles" parameter which doesn't import all font weights.
Here is my array

// Redux version : 3.3.3.5
array(
                        'id'        => 'berry-typography-body-font',
                        'type'      => 'typography',
                        'title'     => __('Global Body Font', 'tymberry-theme-options'),
                        'subtitle'  => __('Specify the body font for using globally through our the whole theme.', 'tymberry-theme-options'),
                        'google'    => true,
                        'all_styles'=> true, // import all google font weights
                        'text-align'=>false,
                        'font-weight' => false,
                        'font-style' => false,
                        'font-size' => false,
                        'line-height' => false,
                        'color' => false,
                        'default'   => array(
                            'font-family'   => 'Raleway, sans-serif'
                        ),
                    ),

Here is the output

Raleway has many font weights here :100,200,300,400,500,600,700,800,900
But the script import only 400.

Could you please help check?

Thanks in advance ;)

[kprovance]

Confirmed, working on it today.

[kprovance]

Fix pushed. Should take care of it. :)

[aonzzung]

Wowwwww!!!!
Thanks for your amazingly quick fix!
;)

[dovy]

If you like Redux, please do one or all of the following:
Write a review: http://wordpress.org/support/view/plugin-reviews/redux-framework
Follow us: http://twitter.com/ReduxFramework
Like us: http://facebook.com/ReduxFramework
Donate: https://www.paypal.com/cgi-bin/webscr?cmd=_s-xclick&hosted_button_id=MMFMHWUPKHKPW

Thanks!

[aonzzung]

Just wrote a comment and gave 5 stars!
Consider to donate also after finishing my project ;)

Thanks!

[dovy]

Thanks!

[aonzzung]

@dovy
I've just updated the latest version 3.3.4.1 but only weight 400 still be imported.
Not all weights are imported. The array is still the same as above.

Am I missing something?

[aonzzung]

Can this feature be tested on localhost?

I checked this line of code and the path ($this->google_json) points to file on my local folder instead of starting with "http://localhost/..."

$fonts = $this->parent->filesystem->execute('get_contents', $this->google_json );

I got $fonts == null and nothing return after "execute".

Any suggestions?

[andreilupu]

@dovy shouldn't the argument "all_styles" be "all-styles"? I've replaced them locally and it kinda works, I didn't do a pull-request cuz I'm not sure of this issue, I still have to dig.

@aonzzung can you try this line in wp-config?

define( 'FS_METHOD', 'direct' );

[dovy]

@aonzzung Just did a push that fixed all the file system issues. It's been confirmed.

Can you verify if the all_styles in not working?

[kprovance]

Reproduced. Working on it, again ;-)