ixbrl-viewer multiple POST thinks its a document set

[michaelchadfleming]

ixbrl-viewer multiple POST thinks its a document set

[austinmatherne-wk]

Thanks, Chad. I'm able to replicate the issue. I suspect that the viewer overriding the keepOpen setting is responsible for the issue here.

@paulwarren-wk can you remind me why that's needed for multi-instance viewers?

[paulwarren-wk]

Thanks, Chad. I'm able to replicate the issue. I suspect that the viewer overriding the keepOpen setting is responsible for the issue here.

@paulwarren-wk can you remind me why that's needed for multi-instance viewers?

It's so that we can do the viewer creation all in one go at the end (on CntlrCmdLine.Filing.End), in order to avoid polluting the controller object with state from our plugin. I think you said that there was precedent for doing this in other plugins, so we should be able to switch to that approach and drop the use of keepOpen.

[austinmatherne-wk]

Hi Chad,

Thanks for the additional details. I may have misunderstood your email. Is the filing in your second request a multi inline document filing, or is it combing the inline documents from both the first and second requests in the response of the second request?

[michaelchadfleming]

HI Austin,

Both input zips are single instance ESMA filings. I'll email over the zips because they're too big for GitHub's 25 MB restriction.

[michaelchadfleming]

Hi Austin, I wasn't able to share via email either. I attached files for the smaller, first filing. Here's a link to the larger filing: https://filings.xbrl.org/filing/969500FU4DRAEVJW7U54-2023-12-31-ESEF-FR-0

output_1.zip
input_1.zip

[austinmatherne-wk]

We specifically support it in ValidateXbrl for validation plugins. We can add something similar to the cntlr, but need to make sure it's cleaned up after the last plugin hooks run (the ValidateXbrl plugin data is cleaned up automatically after validation).

[paulwarren-wk]

That sounds like it should work. I think we would process each document in CntlrCmdLine.Xbrl.Run, and the write out the viewer and cleanup data in CntlrCmdLine.Filing.End.

[austinmatherne-wk]

I opened Arelle/Arelle/pull/1154. Once that's merged ixbrl-viewer should be able to implement and use it's own PluginData dataclass and drop the keepOpen argument.

[michaelchadfleming]

Thank you both for looking into this :)

[paulwarren-wk]

Hang on - we haven't actually done the work on the iXBRL viewer side yet.

We should re-open this issue, but I don't seem to have the permissions to do so.

[michaelchadfleming]

Sorry about that. I reopened the issue.