truth3 / iformbuilder-pdf-generator Goto Github PK

It is a little bit of a mission wen trying to configure more than 3 forms and date ranges, so a simple interface would be great. Honestly that's the only part that's a little time consuming.

The current method of determining the number of records is not reliable and needs to be modified to read the response header keys, rather than the index.

Instead of downloading locally, it would be ideal to store on Google Drive for sharing internally with colleagues.

The README text is not accurate and doesn't clarify the variables that are required very well.

A user request for the following came via email.

Is it possible to add multiple fields into the document name?

Something like:
$pageArray = ["3403833;column1-column2-id;fields=id"]

Right now the PDF files which are saved are named according to the record ID. It will be useful to allow people to download the file with the name based on some data.

Site name and date time could be quite helpful for any inspection based form where a site will have many inspections over time.

If you want to use anything over than the record ID for file names, you run the risk of overwriting files if any of the data is the same.

Consider adding a check to see if the name parameter contains anything other than "id" use a different naming convention which includes ID somewhere in the name to keep everything unique.

There is no warning or indication that the user credentials being used are valid before we start and during. Possible scenarios to catch are

• assignment to forms
• user password
• user locked status

Currently the setup only considers a field value on the parent name which can be used to create the file name. The value may sometimes exist on a subform, so there should be a way to specify that.

When a file name includes system control characters like : or / we run into problems saving the file name. Ideally we should sanitize these to help with system portability.

The record completion count looks at the wrong variable currently to get this value.

Currently if there are special characters in the form label the directory might not get created. We should try to handle these characters better, and at worst fall back to use the table name since it will be database friendly and won't have any trouble being used to name a directory.

The current version doesn't work on forms which have more than 1000 records in them.

Currently this only handles the subdomain of a production iFormBuilder account, and it would be better if different environments can be defined in the keys file.

Existing variable format: apple

Proposed variable format: apple.iformbuilder.com or apple.zerionsandbox.com