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

Alfresco 6 #55

Merged
merged 2 commits into from Oct 23, 2019
Merged

Alfresco 6 #55

merged 2 commits into from Oct 23, 2019

Conversation

douglascrp
Copy link
Contributor

Changes to make the addon compatible with Alfresco 6:
Alfresco SDK upgraded to 4.0-beta1;
Updated itext dependencies to latest version;
Changes to make java code compatible with the new itext version.

Alfresco SDK upgraded to 4.0-beta1;
Updated itext dependencies to latest version;
Changes to make java code compatible with the new itext version.
@douglascrp
Copy link
Contributor Author

This PR would solve the problems for #51 and #53

@digcatcom
Copy link

digcatcom commented Mar 22, 2019

so have tested this pull request, and it allows me to build amps, which can be installed onto 6.0.7-ga.
couple of small things, I added
<project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
to the properties of the repo pom, to get rid of a warning on compile.

also when starting up alfresco after installing, we see ERROR on the transformers its creating for Itext.

content.transformer.iText.TextToPdf.extensions.txt.pdf.priority=50
content.transformer.iText.TextToPdf.extensions.txt.pdf.supported=true
content.transformer.iText.TextToPdf.extensions.csv.pdf.priority=50
content.transformer.iText.TextToPdf.extensions.csv.pdf.supported=true
content.transformer.iText.TextToPdf.extensions.xml.pdf.priority=50
content.transformer.iText.TextToPdf.extensions.xml.pdf.supported=true
content.transformer.iText.TiffToPdf.extensions.tiff.pdf.priority=50
content.transformer.iText.TiffToPdf.extensions.tiff.pdf.supported=true

tested using amps, section uncommented in pom
tested split and rotate
tested on 6.1.2-ga
not sure whether the addon should automatically add these to alfresco-global.properties or not ?

@renatofritola
Copy link

Hello, do you have any forecast for the merge? I am waiting for this improvement to be able to use the addon in alfresco 6.

Thank you.

@digcatcom
Copy link

Hello, do you have any forecast for the merge? I am waiting for this improvement to be able to use the addon in alfresco 6.

Thank you.

Hi renatofritola

Have you tried the PR on a test system? we need more testing to ensure the PR is good and can be merged so any feedback welcome.

@douglascrp douglascrp merged commit 0b1a1a1 into OrderOfTheBee:master Oct 23, 2019
@douglascrp douglascrp deleted the alfresco_6 branch October 23, 2019 19:10
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

3 participants