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

Jump to bottom

Fatal error converting an invoice draft to template #6614

Closed
loxK opened this issue Mar 29, 2017 · 1 comment
Closed

Fatal error converting an invoice draft to template #6614

loxK opened this issue Mar 29, 2017 · 1 comment

Comments

@loxK
Copy link
Contributor

loxK commented Mar 29, 2017

Instructions

Bug

When converting an invoice draft to template I get a PHP fatal error:
Fatal error: Uncaught Error: Class 'FormProjets' not found in /home/lox/public_html/Tests/dolibarr/erp/htdocs/compta/facture/fiche-rec.php on line 892

Environment

  • Version: 5.0.0
  • OS: Ubuntu 16.04.4 x64
  • Web server: Apache/2.4.18 (Ubuntu)
  • PHP: 7.0.15-0ubuntu0.16.04.4
  • Database: MariaDB 5.5.5-10.0.29-MariaDB-0ubuntu0.16.04.1
  • URL(s): compta/facture/fiche-rec.php?facid=1&action=create

Expected and actual behavior

Should create an invoice template, throws a fatal error an the conversion doesn't happen.

Steps to reproduce the behavior

Create an invoice draft click convert to template

selection_928
@AlainRnet
Copy link
Contributor

AlainRnet commented Apr 24, 2017 •
edited

Same problem in 5.0.1
I check in htdocs/compta/facture/fiche-rec.php file, the call to the class, line 38. In line 36, we can see the requirement that projects must be enabled.
Isn't it possible to convert an invoice draft to template without the project's module ?

@atm-florian atm-florian mentioned this issue Apr 24, 2017
Merged
AlainRnet added a commit to AlainRnet/dolibarr that referenced this issue Apr 24, 2017
@AlainRnet
without html.formprojet.class.php called, not possible create invoice…
d9cdaeb 
… template

cf. Dolibarr#6614
without this class AND module Projects disabled, it's not possible to create an invoice template. So I pulled out the call of the class from the condition.
@AlainRnet AlainRnet mentioned this issue Apr 24, 2017
Merged
eldy added a commit that referenced this issue May 4, 2017
@eldy
Merge pull request #6749 from atm-florian/5.0_fix6614
944cd40 
FIX #6614
@eldy eldy closed this as completed in 82c8cee May 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@loxK @AlainRnet