ColdDoc icon indicating copy to clipboard operation
ColdDoc copied to clipboard

Compatibility with open bluedragon 3.0

Open quicken opened this issue 12 years ago • 4 comments

Added a patch for compatibility with open bluedragon 3.0

The recurse and filter options for cfdirectory work differently in openbd. Added a compatibility component which uses a custom method to emulate the same behaviour for cfdirectoy in openbluedragon.

Changed the keys for META_ABSTRACT and META_GENERIC as openbluedragon throws an error when a structure key name contains a colon.

The patch has been tested with openbd 3.0 and Railo 4

quicken avatar Sep 29 '13 12:09 quicken

So unless I'm mistaken, this is a breaking change? As it changes colddoc:abstract -> colddoc_abstract, which would break other people's documentation. Would that be correct?

markmandel avatar Sep 30 '13 00:09 markmandel

Ahh, yes I only just realized that colddoc:abstract is a feature for doing extra annotations. So oops that is correct it's a breaking change.

I guess I really should log a bug report with the open bluedragon guys for what ever reason it's possible to dump the output of the component meta data and the colddoc:abstract keys are show with the correct values in the dump however as soon as they are reference with strFoo["colddoc:abstract"] an exception is thrown. The bluedragon server just does not like a colon in the keyname of the struct returned by the meta functions.

A possibility would be to just wrap the colddoc:abstract definition in another if block that checks for openblueragon.

any thoughts?

Just thought I'd share the code throw it in the bin, use it, does not matter much to me.

cheers

Marcel

On 30/09/2013, at 10:09 AM, Mark Mandel [email protected] wrote:

So unless I'm mistaken, this is a breaking change? As it changes colddoc:abstract -> colddoc_abstract, which would break other people's documentation. Would that be correct?

— Reply to this email directly or view it on GitHub.

quicken avatar Sep 30 '13 02:09 quicken

Unless you want to write a way to switch it out - or better yet - support either option

markmandel avatar Oct 01 '13 04:10 markmandel

Sorry - didn't mean to close it.

markmandel avatar Oct 01 '13 04:10 markmandel