MasterCassidy
MasterCassidy

Reputation: 49

Inconsistent accessibility error: parameter is less accessible than method

I am having an inconsistency issue, I am creating a Csharp updater and I am having a really odd time with one line of code generating this error:

Error Code 1 - Inconsistent Accessibility: Parameter Type 'ModBoxUpdate.ModBoxUpdateXml' is less accessible than method ModBoxUpdate.ModBoxUpdateInfoForm.ModBoxUpdateInfoForm(ModBoxUpdate.IModBoxUpdatable, ModBoxUpdate.ModBoxUpdateXml)'

This is the code i am having the issue with - ModBoxUpdateInfo.cs

using System;
using System.Windows.Forms;

namespace ModBoxUpdate
{
    public partial class ModBoxUpdateInfoForm : Form
    {
                  //This one here//
        public  ModBoxUpdateInfoForm(IModBoxUpdatable applicationInfo, 
        ModBoxUpdateXml updateInfo)
        {
            InitializeComponent();
            if (applicationInfo.ApplicationIcon != null)
                this.Icon = applicationInfo.ApplicationIcon;


        }

    }
}

ModBoxAccept.cs

using System;
using System.Windows.Forms;

namespace ModBoxUpdate
{
    internal partial class ModBoxAcceptForm : Form
    {
        private IModBoxUpdatable applicationInfo;

        private ModBoxUpdateXml updateInfo;

        private ModBoxUpdateInfoForm ModBoxUpdateInfo;
        public ModBoxAcceptForm(IModBoxUpdatable  
        applicationInfo, ModBoxUpdateXml updateInfo)
        {
            InitializeComponent();

            this.applicationInfo = applicationInfo;
            this.updateInfo = updateInfo;

            this.Text = this.applicationInfo.ApplicationName + 
            " - Update Available";


            if (this.applicationInfo.ApplicationIcon != null)
                this.Icon = this.applicationInfo.ApplicationIcon;
            this.NewVersionLabel.Text = string.Format("New Version: {0}", 
            this.updateInfo.Version.ToString());   
        }

    }
}

ModBoxUpdateXml.cs

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Net;
using System.Xml;


namespace ModBoxUpdate
{
    internal class ModBoxUpdateXml
    {
        private Version version;
        private Uri uri;
        private String fileName;
        private String md5;
        private String desc;
        private String launchArgs;
        internal Version Version
        {
            get { return this.version; }
        }
        internal Uri Uri
        {
            get { return this.uri; }
        }
        internal String FileName
        {
            get { return this.fileName; }
        }
        internal String MD5
        {
            get { return this.md5; }
        }
        internal String Description
        {
            get { return this.desc; }
        }
        internal String LaunchArgs
        {

            get { return this.launchArgs; }

        }
        internal ModBoxUpdateXml(Version version, Uri uri, 
        string fileName, string md5, string desc, string launchArgs)
        {
            this.version = version;
            this.uri = uri;
            this.fileName = fileName;
            this.md5 = md5;
            this.desc = desc;
            this.launchArgs = launchArgs;
        }
        internal bool VersionCheck(Version version)
        { 
            return this.version > version;
        }

        internal static bool EOnServer (Uri location)
        {
            try{
            HttpWebRequest Rq = (HttpWebRequest)WebRequest.Create
            (location.AbsoluteUri);

                HttpWebResponse Rp = (HttpWebResponse)Rq.GetResponse();
                Rp.Close();

                return Rp.StatusCode == HttpStatusCode.OK;
            }
            catch   { return false; }
        }

        internal static ModBoxUpdateXml Parse(Uri location, String appID)
        {
            Version version = null;
            string url = "", fileName = "", md5 = "" , 
desc = "" , launchArgs = "";

            try
            {

                XmlDocument doc = new XmlDocument();
                doc.Load(location.AbsoluteUri);

                XmlNode node =doc.DocumentElement.SelectSingleNode
                ("//update[@appId='" + appID + "']");
                if (node == null)
                    return null;
                version = Version.Parse(node["version"].InnerText);
                url = node["url"].InnerText;
                fileName = node["fileName"].InnerText;
                md5 = node["md5"].InnerText;
                desc = node["desc"].InnerText;
                return new ModBoxUpdateXml(version, new Uri(url), 
                fileName, md5, desc, launchArgs);
            }
            catch{return null;}
        }
    }
}

Upvotes: 0

Views: 5450

Answers (1)

Philip Pittle
Philip Pittle

Reputation: 12295

Your ModBoxUpdateInfoForm constructor is public and requires a parameter of type ModBoxUpdateXml which is internal. You get this exception because a caller outside of your assembly couldn't call the public ModBoxUpdateInfoForm constructor, because the caller is not allowed to have knowledge of what a ModBoxUpdateXml is.

Either make ModBoxUpdateXml public, or make the ModBoxUpdateInfoForm constructor internal.

Here is a simple MCVE example:

Causes Compiler Error:

internal class A{}

public class B
{
    public B(A a){}
}

Fix:

//Make this public
public class A{}

public class B
{
    public B(A a){}
}

Or:

internal class A{}

//Make this internal
internal class B
{
    public B(A a){}
}

Or:

internal class A{}

public class B
{
    //Make only the constructor internal
    internal B(A a){}
}

Upvotes: 8

Related Questions