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.

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

Added dynamic configuration of the .pdf metadata with hyperref #42

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

danielvcorreia
Copy link
Member

Since it is not possible to use the fields (author's name, thesis title, etc.) in the matter.tex file (from the cover.tex file), this was the best solution I found to go around that problem.

The alternative was for each template user to change the .pdf metadata parameters to their own. Is that a better solution?

@jpbarraca
Copy link
Contributor

There is an ongoing discussion with the University regarding the inclusion of metadata in the final PDF. Besides the fields you identify, it should also consider a clean version of the abstract.

Regarding your PR, why don't we change the definition of the original macros in order to define the correct values for hyperref, or to create a temporary file which can be used by matter.tex or the Makefile.

@jpbarraca
Copy link
Contributor

The cover.tex file is already very simple, allowing students to define simple variables which are later used. This colides with the fields.txt file you propose and may introduce further confusion.

If another file is required, why not using \write to create it without increasing the complexity for users?

@danielvcorreia
Copy link
Member Author

I can write the variables to a temporary file, but I'm having trouble reading the variables in the preamble of the cover.tex file... It occurs an Emergency stop with the error:

(cannot \read from terminal in nonstop modes)

@danielvcorreia
Copy link
Member Author

The current implementation uses a temporary file to store the information used in the file metadata. It no longer uses the fields.tex file to store that information.

A warning occurred when \\ was used in the metadata parameters, so I changed the \Author command.

Regarding the clean version of the abstract. The current abstract example also uses \\ (warning occurs), and commands like \selectlanguage and \begin{itemize} that originate errors. How do I approach this situation? Should I change the example, or is there a way to clean the text from such commands?

@danielvcorreia
Copy link
Member Author

(I don't know if my previous comment was read)

The current implementation works and uses a temporary file.

I don't know how to approach this:

Regarding the clean version of the abstract. The current abstract example also uses \ (warning occurs), and commands like \selectlanguage and \begin{itemize} that originate errors. How do I approach this situation? Should I change the example, or is there a way to clean the text from such commands?

@jpbarraca

@jpbarraca
Copy link
Contributor

I saw the comment but had no time to compile and check for side effects.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants