Remove UrlGenerator

This CL bites the bullet and removes all traces of the UrlGenerator interface from Shindig, instead favoring the replacement *UriManager classes.

[Paul Lindner, 2010-06-03 04:58:47]

lgtm, perfect time to commit this before 2.0.0 is released.

http://codereview.appspot.com/1496041/diff/1/23
File config/container.js (right):

http://codereview.appspot.com/1496041/diff/1/23#newcode105
config/container.js:105: "gadgets.uri.concat.host" :
"${Cur['defaultShindigTestHost']}",
we support templates here?  interesting....

[henry.saputra, 2010-06-03 17:44:42]

Should this change be included in the UPGRADING release note?

I am sure some people modify/extend the DefaultUrlGenerator or implement their
own UrlGenerator implementation.

- Henry

[Paul Lindner, 2010-06-03 17:55:28]

Yes, keeping UPGRADING up to date should be required for these major
architectural changes.

I know that I'll be making changes based on this CL...

On Jun 3, 2010, at 10:44 AM, henry.saputra@gmail.com wrote:

> Should this change be included in the UPGRADING release note?
> 
> I am sure some people modify/extend the DefaultUrlGenerator or implement
> their own UrlGenerator implementation.
> 
> - Henry
> 
> http://codereview.appspot.com/1496041/show

[johnfargo, 2010-06-03 18:30:25]

On Thu, Jun 3, 2010 at 10:28 AM, <lindner@inuus.com> wrote:

> lgtm, perfect time to commit this before 2.0.0 is released.
>
>
> http://codereview.appspot.com/1496041/diff/1/23
> File config/container.js (right):
>
> http://codereview.appspot.com/1496041/diff/1/23#newcode105
> config/container.js:105: "gadgets.uri.concat.host" :
> "${Cur['defaultShindigTestHost']}",
> we support templates here?  interesting....


I'm pretty sure we do anyway; the test broke for me before I tweaked this
just so ;)


>
>
> http://codereview.appspot.com/1496041/show
>

[johnfargo, 2010-06-03 18:44:26]

Added UPGRADING comments and committed.

The UPGRADING text is fairly sparse, but most people customizing
UrlGenerator should be able to easily provide new impls.

On Fri, Jun 4, 2010 at 12:00 AM, John Hjelmstad <johnfargo@gmail.com> wrote:

> On Thu, Jun 3, 2010 at 10:28 AM, <lindner@inuus.com> wrote:
>
>> lgtm, perfect time to commit this before 2.0.0 is released.
>>
>>
>> http://codereview.appspot.com/1496041/diff/1/23
>> File config/container.js (right):
>>
>> http://codereview.appspot.com/1496041/diff/1/23#newcode105
>> config/container.js:105: "gadgets.uri.concat.host" :
>> "${Cur['defaultShindigTestHost']}",
>> we support templates here?  interesting....
>
>
> I'm pretty sure we do anyway; the test broke for me before I tweaked this
> just so ;)
>
>
>>
>>
>> http://codereview.appspot.com/1496041/show
>>
>
>