Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Minor
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
14 Mar 2011 at 16:59 UTC
Updated:
5 Mar 2012 at 12:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
andras_szilagyi commentedThe module requires the fpdf library added to the sites /sites/all/libraries folder, I have attached a zip.
Comment #2
avpadernoHello, Andras_Szilagyi. You need to create a sandbox project, and report its link here.
Comment #3
andras_szilagyi commentedOk, the sandbox project was created, this is the link: http://drupal.org/sandbox/Andras_Szilagyi/1092174,
thanks for the heads up.
Comment #4
latulipeblanche commentedsubscribing
Comment #5
jaime@gingerrobot.com commentedHi 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.
Comment #6
andras_szilagyi commented@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.
Comment #7
tomdavidson commentedHello, will you being filling out existing PDFs or creating new?
Comment #8
sreynen commentedComment #9
andras_szilagyi commented@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.
Comment #10
andras_szilagyi commented@sreynen
Are you issueing a support request ? How can I help ?
Comment #11
sreynen commentedAndras_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.
Comment #12
andras_szilagyi commented@sreynen
I see, thanks for the help. No im not stopping the project, its just that I am rewriting it into something better.
Comment #13
andras_szilagyi commentedI 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.
Comment #14
andras_szilagyi commentedComment #15
andras_szilagyi commentedI consider my module finished and am closing this issue due to lack of interest by the community.
Comment #16
sreynen commentedHi 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.
Comment #17
Marko B commentedI am very interested in this. Soon i will have this in Live production site. Great module.!
Comment #18
andras_szilagyi commented@deepM
Let me know if I can help wit hany bugfixes or dev, whatever you need.
Comment #19
andras_szilagyi commented@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.
Comment #20
nguyentran commentedHi,
I have just found this module as my demands, i will try this and comment the bugs. Thank a lot for your module.
Comment #21
andras_szilagyi commentedThanks a lot nguyentran