Page Comparison

[09:30:21 CDT(-0500)] <EricDalquist> athena: trying to figure out the best way to associate a IUrlNodeSyntaxHelper with a IStylesheetDescriptor
[09:30:32 CDT(-0500)] <athena> hmm
[09:30:51 CDT(-0500)] <athena> guess for other things like that we've used some sort of key
[09:31:04 CDT(-0500)] <EricDalquist> my first thougth was to add a "urlNodeSyntaxHelperName" to the IStylesheetDescriptor data model
[09:31:09 CDT(-0500)] <athena> yeah
[09:31:10 CDT(-0500)] <EricDalquist> which corresponds to a spring bean-name
[09:31:28 CDT(-0500)] <athena> that seems mostly consistent with what we did for tracking the permission target provider implementation
[09:31:30 CDT(-0500)] <EricDalquist> the only slight weirdness is IStylesheetDescriptor is used to describe both structure and theme XSL
[09:31:40 CDT(-0500)] <EricDalquist> and we would just leave that field null for the theme stylesheets
[09:31:42 CDT(-0500)] <athena> think that one has a registry that tracks implementations by a key name
[09:31:48 CDT(-0500)] <EricDalquist> since those don't really affect the URL
[09:32:07 CDT(-0500)] <athena> so that at least you're getting a service via a key from some other service, which maybe scopes it down a bit from just grabbing spring beans
[09:32:16 CDT(-0500)] <athena> ah, gotcha
[09:33:07 CDT(-0500)] <EricDalquist> right, so that was wondering, is it worth adding some sort of IUrlNodeSyntaxHelper registry?
[09:33:43 CDT(-0500)] <athena> seems like it might be safer . . .
[09:33:47 CDT(-0500)] <EricDalquist> ok
[09:33:57 CDT(-0500)] <athena> though would certainly work the other way
[09:34:17 CDT(-0500)] <EricDalquist> I just don't like having to reference a spring bean from a DB persistent object
[09:34:25 CDT(-0500)] <EricDalquist> but we don't really have a choice here
[09:34:35 CDT(-0500)] <EricDalquist> so maybe the registry is the way to go
[09:34:38 CDT(-0500)] <athena> yeah
[09:34:58 CDT(-0500)] <athena> the registry of course still means you have stuff in the spring config / java referenced from the db
[09:35:07 CDT(-0500)] <athena> but at least it's something more targeted than a bean name
[09:35:08 CDT(-0500)] <EricDalquist> yup