TypeError: Cannot read property '$$phase' of null about ui-tinymce HOT 20 CLOSED

adding in the following code to tinymce.js fixes this issue. No other issues arise.

scope.$on('$destroy', function() {
          if (!tinyInstance) { tinyInstance = tinymce.get(attrs.id); }
          if (tinyInstance) {
            tinyInstance.remove();
            tinyInstance = null;
          }
        });

I would like to see this added to the main branch, as of right now I'll have to provide a modified tinymce.js to my project

from ui-tinymce.

+1 works for me, I am using ui-router views in case that makes any difference. (I've noticed the same error in other Angular plug-ins like Angularstrap tooltip, so maybe ui-router is mis-handling scope?).

from ui-tinymce.

As I have found - it is because tinymce is not properly removed after directive scope is destroyed - so script is trying to do something on destroyed scope(and because of that - there was no $root property in scope).

This was fixed in commit 96d9275

I also had this bug, because of #84. And probably you are also using outdated version because of that.

from ui-tinymce.

Any reason jojobyte's change hasn't been added?

from ui-tinymce.

It would be good to have this fixed soon...

from ui-tinymce.

@Bryan-Turek your solution worked for me, thanks.

from ui-tinymce.

I have the same issue, and @jojobyte's fix works for me.

from ui-tinymce.

@Bryan-Turek fix worked fine for me, but I have the same problem updateing to the next verision with bower as @zxbodya. A new version number would be nice ;-)

from ui-tinymce.

In the meantime I would just modify your tinymce.js, delete the line in bower.json for ui-tinymce (after you bower install it). and modify your .gitignore to keep that path to ui-tinymce. This way it won't be overwritten and you'll always have that modified version. Only downside is, you can't grab the new version, unless you modify that one as well.

from ui-tinymce.

@jojobyte's fix worked for me as well, perhaps this patch should be incorporated?

from ui-tinymce.

Working! Thank you!

from ui-tinymce.

Bug is reproducing for me

from ui-tinymce.

Still reproduces in 0.0.5

from ui-tinymce.

@Uznick did you find a workaround ?

from ui-tinymce.

@mudassir0909 nope, still living with it.

from ui-tinymce.

😢 guess I'll use a forked repo until then

from ui-tinymce.

I switched this to $rootScope, which is what the code should have referenced - this should now be fixed.

from ui-tinymce.

I have the same issue, and @jojobyte's fix works for me.

from ui-tinymce.

@jojobyte fix worked for me. Thanks!

from ui-tinymce.

Manually Triggering the Digest Cycle. Please use like this:
$scope.$apply(() => {
your code here that changes ng-model
});

from ui-tinymce.