[CKEditor.Net] Configuration bug

[michaël acosta]

hi,

I make migration from the ancien .net component to the new official component

i try to change a configuration value " toolBar " on the component function (OnInit, OnLoad, etc).

        protected override void OnInit(EventArgs e)
        {
            base.OnInit(e);
            this.Toolbar = "MyToolBar";
        }
                        

but the value is not save on the config, i have always the FULL toolbar

after debug i found this code

 public CKEditorConfig config
		{
			get
			{
				if (ViewState["CKEditorConfig" + this.ClientID] == null)
					ViewState["CKEditorConfig" + this.ClientID] = new CKEditorConfig(this.BasePath);
				return (CKEditorConfig)ViewState["CKEditorConfig" + this.ClientID];
			}
			set { ViewState["CKEditorConfig" + this.ClientID] = value; }
		}

but this.ClientID is null when the affectation is make on constructor. And value change on the component life.

and after when test if (ViewState+ this.ClientID? == null) the result is true, and the ToolBar value(and other) are overwrite

why not use a private Guid, Created on the Constructor.

private Guid _guidId = new Guid();

public CKEditorControl()
        {
            base.TextMode = TextBoxMode.MultiLine;
            _guidId = Guid.NewGuid();
            
            this.config = new CKEditorConfig(this.BasePath.StartsWith("~") ? this.ResolveUrl(this.BasePath) : this.BasePath);    
        }

public CKEditorConfig config
        {
            get
            {
                if (ViewState["CKEditorConfig" + _guidId] == null)
                    ViewState["CKEditorConfig" + _guidId] = new CKEditorConfig(this.BasePath, Context);
                return (CKEditorConfig)ViewState["CKEditorConfig" + _guidId];
            }
            set { ViewState["CKEditorConfig" + _guidId] = value; }
        }

A Guid ensures is always the same ViewState is used throughout the life of the component.

[kaga]

This issue is related with ticket #7302, after the resolve, it should work fine. I see that you are configure CKEditora in method onInit for the control, I think it is better idea to configure it in the Page_Load.

[kaga]

Could you check the problem still occurs after patching from ticket #7302? If the problem still exists, Could you send me a sample page with the problematic configuration?

[Wiktor Walc]

@silver65b The latest SVN version of the ASP.NET control is available here:
​http://svn.ckeditor.com/CKEditor.NET/trunk/

If you are still able to reproduce the bug using the SVN version, as kaga already suggested, please attach a sample project so that we could reproduce it.

[michaël acosta]

i reproduce the problem with the path.

I have a specific requirement : i encapsulate your component to give at my co-workers a pre-configured component who responding to 99% of cases for the remaining 1% it uses the config file or a parameter in the code.

to do that i need to configure it on constructor because my co-workers can overload in the page load after create component.

i send to you an sample

[michaël acosta]

Little sample solution with the problem.

[kaga]

Thanks for the uploaded example. Yes you are right the problem existed, but now if I change CKEditorConfig.cs CKEditorControl.cs files in your example to the latest version from svn - it seems that it's OK. Please check the newest version CKEditor for ASP.

[Wiktor Walc]

@silver65b - could you confirm that the reported issue is no longer valid? Otherwise please attach a project that is using CKEditor 3.6.1 and where this issue can be still confirmed.

[Wiktor Walc]

This issue is already fixed, as suggested by kaga in previous comment. I'm closing the ticket, however please feel free to reopen it if you are still having this problem in the latest release (3.6.2 at this moment).