Quote from Beer-Me on April 29, 2011, 11:10 am

Great work so far, I think you should add another implementation.

Suggestion 1

You create a file format that map makers will use to create a custom text file that has the relevant map info (Map ID, filename, title, subtitle, picture etc). Therefore each map released could include one of these files.

You then drag + drop that text file, into your application. The app will then parse the new file type, grabbing relevant information. It will then open up the default extras.txt and add the new map data.

Suggestion 2

You create a folder in the Portal 2 directory called Extra Maps, inside of this will store each of the custom map extra files I described earlier. Your app will then loop through all the txt files within that folder and create a new extras.txt based on that information. This is incase you delete your extas.txt file, your app will still be able to generate a new one with the relevant map information.

Quote from Nightgunner5 on April 29, 2011, 11:49 am

I just started it up and was greeted with a fatal error.

Code: Select all

************** Exception Text **************
System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at Portal2ExtrasMod.Form1.Form1_Load(Object sender, EventArgs e)
   at System.Windows.Forms.Form.OnLoad(EventArgs e)
   at System.Windows.Forms.Form.OnCreateControl()
   at System.Windows.Forms.Control.CreateControl(Boolean fIgnoreVisible)
   at System.Windows.Forms.Control.CreateControl()
   at System.Windows.Forms.Control.WmShowWindow(Message& m)
   at System.Windows.Forms.Control.WndProc(Message& m)
   at System.Windows.Forms.ScrollableControl.WndProc(Message& m)
   at System.Windows.Forms.Form.WmShowWindow(Message& m)
   at System.Windows.Forms.Form.WndProc(Message& m)
   at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
   at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
   at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)


Quote from Beer-Me on April 29, 2011, 11:51 am

Do you have the .NET 3.0 runtimes installed?

Quote from Nightgunner5 on April 29, 2011, 12:18 pm

Beer-Me wrote:

Do you have the .NET 3.0 runtimes installed?

Yes, I have .NET 3.0, as well as 3.5 and 4 for that matter.

Quote from Beer-Me on April 29, 2011, 12:34 pm

The error usually occurs when an array is accessed at an invalid index location, making it outside of the index's bounds.

Looking at the source code, the area that it is likely to occur is within:

Code: Select all

   string[] files = Directory.GetFiles(Directory.GetCurrentDirectory(), "*.bsp");
   if (files != null)
   {
      files[0] = Path.GetFileName(files[0]);
      files[0] = files[0].Remove(files[0].Length - 4);
      this.txt_mapfile.Text = files[0];
      this.txt_ID.Text = files[0];
   }


He really needs to check whether files[0] is not empty (null). Hes checking to see if the array is empty, its not but the array elements are.

You should really use if (files.length > 0)

Quote from Haapavuo on April 29, 2011, 1:08 pm

Beer-Me wrote:

The error usually occurs when an array is accessed at an invalid index location, making it outside of the index's bounds.

Looking at the source code, the area that it is likely to occur is within:

Code: Select all

   string[] files = Directory.GetFiles(Directory.GetCurrentDirectory(), "*.bsp");
   if (files != null)
   {
      files[0] = Path.GetFileName(files[0]);
      files[0] = files[0].Remove(files[0].Length - 4);
      this.txt_mapfile.Text = files[0];
      this.txt_ID.Text = files[0];
   }


He really needs to check whether files[0] is not empty (null). Hes checking to see if the array is empty, its not but the array elements are.

You should really use if (files.length > 0)

Thanks for your replies! I think I've fixed it now.

I will try to make the implementations you mentioned earlier...

Quote from Nightgunner5 on April 29, 2011, 2:50 pm

It would be cool if it was possible to click and drag a file like this:

Code: Select all

//include_maps:n5_sequential01.bsp:n5_sequential02.bsp:n5_sequential03.bsp
//include_materials/vgui/extras:n5_sequential01.vmt:n5_sequential01.vtf
   "n5_sequential01"
   {
      "title"      "Sequential"
      "subtitle"   "A custom map by Nightgunner5"
      "pic"      "vgui/extras/n5_sequential01"
      "map"      "n5_sequential01"
   }

Onto your program and have it automatically install everything that is needed.

Quote from Haapavuo on April 29, 2011, 3:26 pm

Nightgunner5 wrote:

It would be cool if it was possible to click and drag a file like this:

Code: Select all

//include_maps:n5_sequential01.bsp:n5_sequential02.bsp:n5_sequential03.bsp
//include_materials/vgui/extras:n5_sequential01.vmt:n5_sequential01.vtf
   "n5_sequential01"
   {
      "title"      "Sequential"
      "subtitle"   "A custom map by Nightgunner5"
      "pic"      "vgui/extras/n5_sequential01"
      "map"      "n5_sequential01"
   }

Onto your program and have it automatically install everything that is needed.

I'll try to make that possible.

What are those commented lines? Are those important?

Quote from Nightgunner5 on April 29, 2011, 3:28 pm

Haapavuo wrote:

Nightgunner5 wrote:

It would be cool if it was possible to click and drag a file like this:

Code: Select all

//include_maps:n5_sequential01.bsp:n5_sequential02.bsp:n5_sequential03.bsp
//include_materials/vgui/extras:n5_sequential01.vmt:n5_sequential01.vtf
   "n5_sequential01"
   {
      "title"      "Sequential"
      "subtitle"   "A custom map by Nightgunner5"
      "pic"      "vgui/extras/n5_sequential01"
      "map"      "n5_sequential01"
   }

Onto your program and have it automatically install everything that is needed.

I'll try to make that possible.

What are those commented lines? Are those important?

That's actually a working configuration file for my patcher. The commented lines tell it files to copy.