Comments on: CDN Rewrites http://www.phoenixheart.net phoenix.heart - portfolio & more Wed, 10 Mar 2010 00:26:01 +0000 http://wordpress.org/?v=2.9.2 hourly 1 By: Artem Russakovskiihttp://www.phoenixheart.net/wp-plugins/cdn-rewrites/comment-page-1/#comment-8537 Artem Russakovskii Sun, 17 Jan 2010 08:25:16 +0000 http://www.phoenixheart.net/?page_id=321#comment-8537 An, I see your problem with .live(). Well, looks like 2.8 was the first one with 1.3, while 2.7 still had 1.2. You can easily check by looking at svn revisions: http://svn.automattic.com/wordpress/tags/2.8/wp-includes/js/jquery/jquery.js vs http://svn.automattic.com/wordpress/tags/2.7/wp-includes/js/jquery/jquery.js I think if you specify 2.8 as the min revision for your plugin, you should be able to then use wp_enqueue_script() and load the script correctly. For the rewrites, I haven't played with your code yet. I will when I have some free time. Thanks, Artem An,
I see your problem with .live(). Well, looks like 2.8 was the first one with 1.3, while 2.7 still had 1.2. You can easily check by looking at svn revisions: http://svn.automattic.com/wordpress/tags/2.8/wp-includes/js/jquery/jquery.js vs http://svn.automattic.com/wordpress/tags/2.7/wp-includes/js/jquery/jquery.js
I think if you specify 2.8 as the min revision for your plugin, you should be able to then use wp_enqueue_script() and load the script correctly.
For the rewrites, I haven’t played with your code yet. I will when I have some free time.
Thanks,
Artem

]]> By: phoenix.hearthttp://www.phoenixheart.net/wp-plugins/cdn-rewrites/comment-page-1/#comment-8536 phoenix.heart Sun, 17 Jan 2010 08:01:52 +0000 http://www.phoenixheart.net/?page_id=321#comment-8536 Strange enough Artem, I don't see your comment in either approved, spam, or pending. I read your tickets on WP forum though, and wanted to says thanks :)You're correct about including the plugin js and css on any admin page, it's a bad habit as specified by Ozh some time ago. I will definitely keep this in mind for the next version.For jQuery, thing is my plugin(s) use live() which is only available from 1.3 when earlier WP versions (not sure with 2.9) only come with up to 1.2.x (please correct me if I'm wrong). If there is a way to force WordPress to use 1.3.2↑ I'll much appreciate.For CDN Rewrites not working on your site, to admit I don't have any clue yet. Help me with a bit more details will you? Strange enough Artem, I don’t see your comment in either approved, spam, or pending.
I read your tickets on WP forum though, and wanted to says thanks :)

You’re correct about including the plugin js and css on any admin page, it’s a bad habit as specified by Ozh some time ago. I will definitely keep this in mind for the next version.

For jQuery, thing is my plugin(s) use live() which is only available from 1.3 when earlier WP versions (not sure with 2.9) only come with up to 1.2.x (please correct me if I’m wrong). If there is a way to force WordPress to use 1.3.2↑ I’ll much appreciate.

For CDN Rewrites not working on your site, to admit I don’t have any clue yet. Help me with a bit more details will you?

]]> By: Artem Russakovskiihttp://www.phoenixheart.net/wp-plugins/cdn-rewrites/comment-page-1/#comment-8535 Artem Russakovskii Sun, 17 Jan 2010 07:29:46 +0000 http://www.phoenixheart.net/?page_id=321#comment-8535 An, did you delete my comment or it's stuck in moderation? I filed a few tickets, please check the wordpress plugin page. An, did you delete my comment or it’s stuck in moderation?
I filed a few tickets, please check the wordpress plugin page.

]]> By: CDN Rewrites Plugin for WordPress - datenschwanz.nethttp://www.phoenixheart.net/wp-plugins/cdn-rewrites/comment-page-1/#comment-7795 CDN Rewrites Plugin for WordPress - datenschwanz.net Thu, 10 Dec 2009 16:30:11 +0000 http://www.phoenixheart.net/?page_id=321#comment-7795 [...] destroying page load performance. I have opted for rewriting the static content references with the CDN Rewrites WordPress plugin. It almost got the job done. I didn’t want it rewriting anything for [...] [...] destroying page load performance. I have opted for rewriting the static content references with the CDN Rewrites WordPress plugin. It almost got the job done. I didn’t want it rewriting anything for [...]

]]> By: Sin tiempo para escribir.27, Carrerohttp://www.phoenixheart.net/wp-plugins/cdn-rewrites/comment-page-1/#comment-6690 Sin tiempo para escribir.27, Carrero Fri, 16 Oct 2009 23:12:54 +0000 http://www.phoenixheart.net/?page_id=321#comment-6690 [...] CDN Rewrites. En inglés. vía: phoenix.heart [...] [...] CDN Rewrites. En inglés. vía: phoenix.heart [...]

]]> By: Martinhttp://www.phoenixheart.net/wp-plugins/cdn-rewrites/comment-page-1/#comment-6670 Martin Fri, 16 Oct 2009 07:21:33 +0000 http://www.phoenixheart.net/?page_id=321#comment-6670 #1, yeah maybe it is a conflict. I mocked around and I think if I removed the CDN jquery files the upload dialog would work again - so maybe there's a jquery conflict with the WP jquery files !? But I guess if it works for you with WP 2.8.4 it's more likely to be a plugin conflict... #1, yeah maybe it is a conflict. I mocked around and I think if I removed the CDN jquery files the upload dialog would work again – so maybe there’s a jquery conflict with the WP jquery files !? But I guess if it works for you with WP 2.8.4 it’s more likely to be a plugin conflict…

]]> By: phoenix.hearthttp://www.phoenixheart.net/wp-plugins/cdn-rewrites/comment-page-1/#comment-6665 phoenix.heart Fri, 16 Oct 2009 02:07:05 +0000 http://www.phoenixheart.net/?page_id=321#comment-6665 <p>Martin, I agree with you on "strictly speaking". But strictly speaking too, wp_header and wp_footer are mandatory, since almost all javascript, css includes, and a lot of other tasks are triggered via these two functions. So the themes should take them into account instead of them taking some specific themes IMHO.</p> <p>About the bugs:</p> <p>#1. I will certainly check again. However that's not necessarily mean it's a CDN Rewrites bug - sometimes it may be a plugin conflict.</p> <p>#2. This is A BUG! I forgot to declare t as global. Thank you!</p> <p>Cheers,</p> <p>An</p> Martin, I agree with you on “strictly speaking”. But strictly speaking too, wp_header and wp_footer are mandatory, since almost all javascript, css includes, and a lot of other tasks are triggered via these two functions. So the themes should take them into account instead of them taking some specific themes IMHO.

About the bugs:

#1. I will certainly check again. However that’s not necessarily mean it’s a CDN Rewrites bug – sometimes it may be a plugin conflict.

#2. This is A BUG! I forgot to declare t as global. Thank you!

Cheers,

An

]]> By: Martinhttp://www.phoenixheart.net/wp-plugins/cdn-rewrites/comment-page-1/#comment-6661 Martin Thu, 15 Oct 2009 22:40:30 +0000 http://www.phoenixheart.net/?page_id=321#comment-6661 @phoenix.heart according to the codex: "This hook is theme-dependant which means that it is up to the author of each WordPress theme to include it. It may not be available on all themes, so you should take this into account when using it." So strictly speaking the plugin should account for it. I think I might have discovered another few bugs. #1 I suddenly wasn't able to invoke the media upload dialog - clicking the icon it darkens the screen and then nothing happens. Disabling CDN and it works again. #2 In the goTop() function in the admin.js file. t is undefined if clearTimeout() is called. @phoenix.heart according to the codex: “This hook is theme-dependant which means that it is up to the author of each WordPress theme to include it. It may not be available on all themes, so you should take this into account when using it.”
So strictly speaking the plugin should account for it.
I think I might have discovered another few bugs.
#1
I suddenly wasn’t able to invoke the media upload dialog – clicking the icon it darkens the screen and then nothing happens. Disabling CDN and it works again.
#2
In the goTop() function in the admin.js file. t is undefined if clearTimeout() is called.

]]> By: Mikehttp://www.phoenixheart.net/wp-plugins/cdn-rewrites/comment-page-1/#comment-6618 Mike Wed, 14 Oct 2009 03:06:13 +0000 http://www.phoenixheart.net/?page_id=321#comment-6618 You can see this plug in work without actually having a CDN. Setup an A-Record for your domain like  images.domain.com Then use the plugin to rewrite all your images to be images.domain.com You can right click on one of the images and see the source URL.  Assuming you have a CDN, you could CNAME images.domain.com and get the same result - Mike Colburn You can see this plug in work without actually having a CDN.
Setup an A-Record for your domain like  images.domain.com
Then use the plugin to rewrite all your images to be images.domain.com
You can right click on one of the images and see the source URL.  Assuming you have a CDN, you could CNAME images.domain.com and get the same result
- Mike Colburn

]]> By: phoenix.hearthttp://www.phoenixheart.net/wp-plugins/cdn-rewrites/comment-page-1/#comment-6617 phoenix.heart Wed, 14 Oct 2009 02:41:03 +0000 http://www.phoenixheart.net/?page_id=321#comment-6617 Martin, it's not a bug. Calling wp_footer or wp_header is required for almost every plugin, as the functions trigger the plugin actions. If a theme doesn't call wp_footer or wp_header, then I can tell you it's a sucking theme. Martin, it’s not a bug. Calling wp_footer or wp_header is required for almost every plugin, as the functions trigger the plugin actions.
If a theme doesn’t call wp_footer or wp_header, then I can tell you it’s a sucking theme.

]]>