The scope of this project is to offer the missing pdf views export type. I have browsed all views exports and feeds and have not found an export to pdf for the views results. I look forward to your review and suggestions. this module is allready used in some projects for some of my clients that requested this.

thank you in advance!

CommentFileSizeAuthor
#5 screenshot.png154.03 KBjaime@gingerrobot.com
#1 fpdf.zip201.22 KBandras_szilagyi

Comments

andras_szilagyi’s picture

StatusFileSize
new201.22 KB

The module requires the fpdf library added to the sites /sites/all/libraries folder, I have attached a zip.

avpaderno’s picture

Status: Active » Postponed (maintainer needs more info)

Hello, Andras_Szilagyi. You need to create a sandbox project, and report its link here.

andras_szilagyi’s picture

Status: Postponed (maintainer needs more info) » Needs review

Ok, the sandbox project was created, this is the link: http://drupal.org/sandbox/Andras_Szilagyi/1092174,
thanks for the heads up.

latulipeblanche’s picture

subscribing

jaime@gingerrobot.com’s picture

Category: task » support
StatusFileSize
new154.03 KB

Hi Andras_Szilagyi,

I installed your project from git.

My comments so far:

* A README.txt file will help if people install from drush, to know to get the fpdf module
* Also some guidance of settings up a pdf in the views feed could also go in the README.txt as I didn't see the message at the top of the views page right away
* When I set the feed to be pdf I get:
--
* warning: Parameter 3 to views_ui_build_form_state() expected to be a reference, value given in /home/jaime/WEB/gingerrobot.com/projects/community/vpe/sites/all/modules/views/includes/admin.inc on line 1606.
* warning: call_user_func_array() expects parameter 1 to be a valid callback, no array or string given in /home/jaime/WEB/gingerrobot.com/projects/community/vpe/includes/form.inc on line 378.
* warning: Invalid argument supplied for foreach() in /home/jaime/WEB/gingerrobot.com/projects/community/vpe/sites/all/modules/views/includes/admin.inc on line 1542.
* warning: Attempt to assign property of non-object in /home/jaime/WEB/gingerrobot.com/projects/community/vpe/sites/all/modules/views/includes/admin.inc on line 1613.
* Display "Feed" uses a path but the path is undefined.

* The error message I get after this when I set up the feed for my view is:

warning: require(): Filename cannot be empty in sites/all/modules/views_pdf_export/views_pdf_export.theme.inc on line 14.

However In the settings it does have a file name set as in screenshot.

Hope this helps,

Jaime.

andras_szilagyi’s picture

Status: Needs review » Needs work

@jaimekristene
Thank you for your competent review, the info is great.
I will have a look at those issues and implement your feedback, I also want to add more settings to make it even more configurable.
I'm changing the status until I'm done with the changes.

@latulipeblanche
Thanks for your support,
it will be ready soon, and it wont let you down.

tomdavidson’s picture

Hello, will you being filling out existing PDFs or creating new?

sreynen’s picture

Category: support » task
andras_szilagyi’s picture

@tomdavidson

The pdfs will be generated, so new.

Last month a sandbox project made it through as a full module (http://drupal.org/project/views_pdf) i havent tested it yet, but i'm sure i will. Its supposed to do the same thing. Have a look at it.

andras_szilagyi’s picture

@sreynen

Are you issueing a support request ? How can I help ?

sreynen’s picture

Andras_Szilagyi, this issue is an application to create a full project. All full project applications are in the "task" category, so I changed the category. It sounds like you're considering stopping your project and working on http://drupal.org/project/views_pdf instead. If you decide to do that, please close this issue.

andras_szilagyi’s picture

@sreynen

I see, thanks for the help. No im not stopping the project, its just that I am rewriting it into something better.

andras_szilagyi’s picture

I finished the module review, the idea was to change the approach by making use of the excelent work done on the print_pdf module (not reinventing the wheel). This ensures a smaller, better more reliable codebase.

The new version is basically a bridge between the print_pdf module (from "Printer-friendly pages") and the views module. It was tested one 2 different drupal 6.20 installations.

Could someone review my module, I could relay use the feedback. Thanks in advance.

andras_szilagyi’s picture

Status: Needs work » Needs review
andras_szilagyi’s picture

Component: new project application » module
Priority: Normal » Minor
Status: Needs review » Closed (fixed)

I consider my module finished and am closing this issue due to lack of interest by the community.

sreynen’s picture

Hi Andras_Szilagyi,

I wouldn't say there's lack of interest so much as lack of time. There are hundreds of applications and they each take time to review, with only a few regular volunteer reviewers. We're in ongoing discussions about how to speed up this process, but it's still much slower than we'd like it to be. If the lack of response is the only reason you closed this, please consider re-opening it and someone will get to it eventually, if not soon. If you have other reasons for closing it, that's fine, I just wanted to make sure you don't misinterpret the lack of response.

Marko B’s picture

I am very interested in this. Soon i will have this in Live production site. Great module.!

andras_szilagyi’s picture

@deepM

Let me know if I can help wit hany bugfixes or dev, whatever you need.

andras_szilagyi’s picture

@sreynen

Thanks for the reply, I guess I'll try to contribute by testing other peoples work from now on, since there's lots of development but not enough testing going on.

nguyentran’s picture

Hi,
I have just found this module as my demands, i will try this and comment the bugs. Thank a lot for your module.

andras_szilagyi’s picture

Thanks a lot nguyentran