[Plugin: Scripts n Styles] Consider revising settings page.

  • Resolved Sterling Hamilton

    (@sterlo)


    12 years, 7 months ago

    Currently, in your plugin there’s this line:

    In the usage section.

    `$all_posts = get_posts( array( ‘numberposts’ => -1, ‘post_type’ => ‘any’, ‘post_status’ => ‘any’ ) );

    I work with sites with 5000+ posts, this line ends up needing more than 256 MB of memory.

    I would recommend either removing the usage section OR finding a way to make smaller requests.

    With a site that has 5000+ posts, plus media assets, other plugins with post types that query is HUGE.

    I’ve had to actually comment it out – I really like using the plugin and I didn’t want that line to be the reason I didn’t use it.

    http://wordpress.org/extend/plugins/scripts-n-styles/

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author WraithKenny

    (@wraithkenny)

    12 years, 6 months ago

    I am working on this, but I got a bit stalled do to high work load (at work). If you’d like to check out the beta at https://github.com/WraithKenny/Scripts-n-Styles any feed back would be really appreciated 🙂 You can drop comments on my blog page also, or DM me on twitter (@WraithKenny) with feedback.

    The current idea is to put it on it’s own tab/page. I’ve got a better uninstall loop going for it too. The new query uses the metakey parameters for (I’m hoping) better proformance in general. The beta also has some nifty ajax going on too.

    Also, where would you like the menus to go? I’m stuck as to where the best place for that is… I’m probably just going to make a settings page where you can select an option on where to put the settings page links. Some of the options sound like “appearance” while others make more sense in “options” while the thing as a whole sounded like “tools” but people sometimes have strong feels about their menus.

    Plugin Author WraithKenny

    (@wraithkenny)

    12 years, 6 months ago

    I’ll try to either get the 3 version done sometime soon (within a week), or update the 2 branch to disable the feature.

    Plugin Author WraithKenny

    (@wraithkenny)

    12 years, 6 months ago

    I released an update (2.0.4) to solve your issue! In the 3.x versions, it’ll have a much more robust sollution 🙂

    Thread Starter Sterling Hamilton

    (@sterlo)

    12 years, 6 months ago

    Thanks! I’ll check it out when I get a lul in the work load here at the office ^_^

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘[Plugin: Scripts n Styles] Consider revising settings page.’ is closed to new replies.