Import recipe from Brewmate causes Brewtarget to stop working

[Jay]

OS: Win7 32-bit Ultimate SP1
BrewTarget - 2.0.1

Default Install into Program Files directory. Set Brewtarget.exe to run as Adminstrator

Run Brewtarget and choose import recipe from the File menu.

Navigate to Brewmate recipe sub directory and click on recipe xml file

Error pops up from Brewtarget saying Brewtarget has stopped working. Below is the details from the error:

Problem signature:
  Problem Event Name: APPCRASH
  Application Name: brewtarget.exe
  Application Version: 0.0.0.0
  Application Timestamp: 5137df2c
  Fault Module Name: QtCore4.dll
  Fault Module Version: 4.7.4.0
  Fault Module Timestamp: 4e5f8d4e
  Exception Code: c0000005
  Exception Offset: 0010d542
  OS Version: 6.1.7601.2.1.0.256.1
  Locale ID: 1033
  Additional Information 1: 0a9e
  Additional Information 2: 0a9e372d3b4ad19135b953a78882e789
  Additional Information 3: 0a9e
  Additional Information 4: 0a9e372d3b4ad19135b953a78882e789

BrewTarget should import the recipe xml file from BrewMate.

[Mik Firestone (mikfire)]

I'm not familiar with brewmate.

Does the recipe.xml file contain all of your recipes, or just one? Is there some way you could provide me with the recipe.xml file (dropbox, email, etc.) so I can test with it?

[Mik Firestone (mikfire)]

Wow. That's an impressive core dump. Okay. I have what I need and will start working on it.

[Mik Firestone (mikfire)]

I will fix the core dump, because we should handle it better than that, but the real problem is with the brewmate XML file. According to the BeerXML specification, the STYLE record set has 15 required attributes. Since these are required, we don't check to see if they are present or not.

It looks like brewmate didn't implement the specification, since the XML file only shows the style name and is missing the other required attributes. This appears to be the case with both the recipes provided with the software and the recipes I created.

[Jay (jbrend)]

Thanks Mike -

I can add the XML file from Brewmate if that would help?

[Mik Firestone (mikfire)]

Bug fix has been committed with merge request 54.