When a page includes multiple nodes with CPN CSS and/or JS, only one of those nodes JS/CSS gets attached.

Comments

lee20’s picture

Status: Active » Needs review
StatusFileSize
new1.18 KB

Attached is a patch that solves the problem.

davidtrainer’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new1.16 KB

I confirmed that the above patch works, and rerolled it from within the cpn module directory with no substantial changes.

damienmckenna’s picture

+1.

damienmckenna’s picture

Status: Reviewed & tested by the community » Needs review
StatusFileSize
new1.66 KB

This needed to be rerolled. Also, I changed the per-content-type code to have the variable named after the content type instead of the nid, which seemed redundant and may help avoid the same file(s) being loaded multiple times.

damienmckenna’s picture

Status: Needs review » Fixed

Committed, thanks everyone.

damienmckenna’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Fixed » Patch (to be ported)

Needs to be backported.

damienmckenna’s picture

Issue tags: +Needs backport to D6

Tag.

damienmckenna’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Patch (to be ported) » Fixed
Issue tags: -Needs backport to D6

This isn't needed for the D6 branch.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.