Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: grab the sheetData for the actor #117

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
+17 −3
Open

feat: grab the sheetData for the actor #117

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6356246
feat: grab the sheetData for the actor so that it can be used in a ma…
arcos Dec 13, 2023
ec8093e
Delete sheet data when no longer needed
arcos Dec 13, 2023
888889b
Removed an unneeded console.log
arcos Dec 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 17 additions & 3 deletions scripts/pdfsheet.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,6 @@ Hooks.on("getActorSheetHeaderButtons", (sheet, buttons) => {
// added pc for cypher system
// TODO: have to refactor this with something generic
if (!["character", "PC", "Player", "npc", "pc"].includes(sheet.actor.type ?? sheet.actor.data.type)) return;

buttons.unshift({
label: "Export to PDF",
class: "export-pdf",
Expand All @@ -276,6 +275,11 @@ class Pdfconfig extends FormApplication {
/** The module's ID */
static ID = "pdf-sheet";

close() {
delete this.actor.sheetData;
return super.close();
}

/** @override */
static get defaultOptions() {
return mergeObject(super.defaultOptions, {
Expand Down Expand Up @@ -319,7 +323,7 @@ class Pdfconfig extends FormApplication {
}

/** Create a form with inputs for each PDF field and fill them in with Actor data */
createForm(buffer) {
async createForm(buffer) {
const inputForm = document.getElementById("fieldList");

// Empty input form
Expand All @@ -333,15 +337,22 @@ class Pdfconfig extends FormApplication {
// Get Actor data
const actor = this.actor;

if(typeof this.actor.sheet.getData === "function"){
const sheetData = await this.actor.sheet.getData();
actor.sheetData = sheetData;
}

// Begin grouping logs
console.group("PDF Sheet");
// Log Actor Data
console.log("Actor Data:", actor);
// Log all PDF fields
console.log("PDF fields:", pdfFields);



// Get mapping from settings
let mapping = ""
let mapping = "";

if (["character", "PC", "Player", "pc"].includes(actor.type ?? actor.data.type)) {
console.log("got mapping for PC")
Expand All @@ -365,6 +376,7 @@ class Pdfconfig extends FormApplication {
// Return as evaluated JavaScript with the actor as an argument
mapping = Function(`"use strict"; return function(actor) { return ${mapping} };`)()(actor);
} catch (err) {
delete this.actor.sheetData;
// End console group
console.groupEnd();
// Close the application
Expand Down Expand Up @@ -499,6 +511,7 @@ class Pdfconfig extends FormApplication {
const filled_pdf = pdfform(minipdf).transform(buffer, fields);

const blob = new Blob([filled_pdf], { type: "application/pdf" });

saveAs(blob, `${this.actor.name ?? "character"}.pdf`);
}

Expand All @@ -509,5 +522,6 @@ class Pdfconfig extends FormApplication {

document.getElementById("pdf-header").setAttribute("style", "display: none");
document.getElementById("pdf-export").style.display = "block";
delete this.actor.sheetData;
}
}