Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#12353 closed defect (fixed)

dojo._postLoad not set when RichText.js is loaded via AMD

Reported by: Adam Peller Owned by: Rawld Gill
Priority: high Milestone: 1.6
Component: Editor Version: 1.6.0rc1
Keywords: Cc: Kris Zyp, ben hockey, Rawld Gill, chuckd@…
Blocked By: Blocking:

Description

Chuck Dumont reports needing to add dojo._postLoad=true to backCompat.js in order to satisfy dijit/_editor/RichText.js. Perhaps it's an issue of the addOnLoad handlers firing out of order? He was using requirejs

Attachments (1)

backCompat.patch (406 bytes) - added by Adam Peller 9 years ago.
patch from Chuck Dumont (IBM, CCLA)

Download all attachments as: .zip

Change History (6)

Changed 9 years ago by Adam Peller

Attachment: backCompat.patch added

patch from Chuck Dumont (IBM, CCLA)

comment:1 Changed 9 years ago by ben hockey

Cc: Kris Zyp ben hockey Rawld Gill chuckd@… added
Keywords: kzyp neonstalwart rcgill chuckd@us.ibm.com removed

comment:2 Changed 9 years ago by chuckd

Looking at this again, I think the real bug is a typo in the addOnLoad handler (line 68):

dojo.postLoad = dojo.config.afterOnLoad = true;

should instead be

dojo._postLoad = dojo.config.afterOnLoad = true;

comment:3 Changed 9 years ago by Adam Peller

Owner: set to Rawld Gill

comment:4 Changed 9 years ago by Rawld Gill

Resolution: fixed
Status: newclosed

(In [23949]) fixes #12353, !strict

comment:5 Changed 9 years ago by Adam Peller

Milestone: tbd1.6

Good catch, Chuck. We can consider putting this in the 1.6 branch also.

Note: See TracTickets for help on using tickets.