Reputation: 49
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
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