search

chilli-axe / mpc-autofill

Print automation software (leveraging MakePlayingCards) for your tabletop gaming community
https://mpcautofill.github.io
GNU General Public License v3.0
220 stars 97 forks source link

CSV format does not populate deck list #222

Closed Kynazeras closed 8 months ago

Kynazeras commented 8 months ago

System

Affected Components

Description

When following the format for .csv files, the application is not able to populate a double-sided deck If I use this format:

Quantity, Front, Front ID, Back, Back ID
1,Ancient Tomb,     ,Ancient Tomb,

I get a blank card and the network tab tells me that the application issued a search for "1" which looks like it is searching the quantity as the card name

The expected output in the above case would be a single card with both front and back being Ancient Tomb

Steps to Reproduce

  1. Make a .csv file following the official template (using blanks for Front ID and Back ID vs using a string had no effect on the results)
  2. Import that .csv into the tool
  3. See that blank cards have been populated
ndepaola commented 8 months ago

g'day - thanks for the detailed bug report! looks like the issue here is the csv parsing library i'm using expects headers to be exactly separated by the delimiter , & doesn't handle spaces between them - so Quantity,Front,Front ID,Back,Back ID is valid but Quantity, Front, Front ID, Back, Back ID is not

obviously i'll fix this bug but at least removing the spaces between your column headers is a workable solution for the time being :)