Download & Extend
Simple CDN » Issues

Merge Simple CDN and CDN modules: migration path

Posted by Wim Leers on February 3, 2011 at 11:50am
Project:Simple CDN
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:critical
Assigned:Unassigned
Status:active

Issue Summary

The Parallel module will be merged with the CDN module #933942: Merge Parallel and CDN modules: migration path. Less CDN modules for Drupal will make at least this aspect of Drupal easier. The CDN module also already has a Drupal 7 release, and uses the new API functionality in Drupal 7 to no longer require a Drupal core patch. The 2.0 version for Drupal 6 also no longer *requires* a core patch: it can work without it, but with suboptimal performance and not all file URLs being altered.

The Simple CDN module doesn't support all file URLs to be altered anyway, so that's a non-issue. Out-of-the-box, it only supports Image and Image Cache URLs. These are also supported by the CDN module, even without the core patch.

The Simple CDN module also provides an API to allow module developers to specify specific CDN URLs. This can also be achieved *without* custom code through the UI of the CDN module: hence less code and less dependencies. And in case you've got a very advanced set-up, the CDN module provides a hook through which you can define *any* logic you would like to determine which CDN to use.

So, what do you think?

Login or register to post comments

Comments

#1

Posted by jdelaune on February 3, 2011 at 7:35pm

You could have sold it to me after one word. Lets close this baby down.

#2

Posted by Wim Leers on February 4, 2011 at 8:42am

Hah :)

What do you propose?

#3

Posted by jdelaune on February 5, 2011 at 12:07am

I assume we can write a function for cdn module which extracts settings from simplecdn if it's installed.

I don't know how CDN works but I can give you pull origin url's and which elements it's meant to be used on e.g. images, imagecache etc.

Or we can put one in simplecdn which inserts that data into CDN.

#4

Posted by Wim Leers on February 5, 2011 at 1:47pm

You mean that the Simple CDN module can provide an upgrade path towards the CDN module, right? An upgrade path that transforms Simple CDN's settings to CDN's format?

#5

Posted by jdelaune on February 6, 2011 at 12:59am

yup

#6

Posted by Wim Leers on February 13, 2012 at 6:49pm

Let's make this happen? :)

#7

Posted by Wim Leers on April 23, 2012 at 9:24am
Priority:normal» critical

If you can work on providing the upgrade path, I'd be happy to accommodate you with whatever you need the CDN module to do. Let's move this forward…

nobody click here