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

V1.6.0 #680

wants to merge 43 commits into
base: master


None yet
9 participants
Copy link

M4rt1n17 commented Dec 19, 2018

Issue Type (Please check one or more)

  • Bugfix
  • Improvement of an existing Feature
  • New Feature

Could you pls support PHP 7.3. Guess thats not a big thing to do, as all other webapps (which supported PHP 7.2) are able to run under PHP 7.3

Kovah and others added some commits Sep 18, 2017

Add files via upload
Log logins.
To be able to block brute-force attacks with fail2ban.
Update Mdl_custom_fields.php
This fixes the 500 error on php5.6
Assign all clients to a user account (IP-582)
Cuando se añade una cuenta de usuario de tipo invitado. Al añadir los clientes que puede ver, aparece una nueva opción que nos permite añadir todos los clientes automáticamente. También cuando se cree un nuevo cliente este se añadira a todos las cuentas de usuario que tengan esta opción habilitada. Muy útil para cuentas de usuario que usarán en contabilidad.


When a user type user account is added. When you add the clients you can see, a new option appears that allows us to add all clients automatically. Also when a new client is created this will be added to all user accounts that have this option enabled. Very useful for user accounts that will be used in accounting.
Edit ip_lang
Corregida la frase de la variable user_all_clients_text


Corrected the phrase of the variable user_all_clients_text
Make $custom_fields available in public templates
The same custom fields will be available as what is available in a PDF

Forum topic:
Possible fix for IP-577
As $mdl_invoice_tax_rates is not available in the copy_invoice function we load the corresponding model first. Couldn't reproduce the error but this should work.
Fix escaping for create quote/invoice modals (IP-606)
+ various smaller code corrections
Merge pull request #564 from InvoicePlane/v1.5.6
Pull changes from v1.5.6
Merge pull request #572 from laemol/IP-612
IP-612 Fixed Mollie payments
Merge pull request #587 from InvoicePlane/v1.5.6
Update 1.6.0 from 1.5.6

cornelinux and others added some commits Jan 28, 2018

Add another quote state "invoiced"
The quote state indicates, that for an accepted quote
an invoice was written.
When an invoiced is created from a quote the
state of the quote gets changed automatically.

Closes IP-636
Merge branch 'master' into v1.6.0
# Conflicts:
#	application/modules/guest/controllers/Payment_handler.php
Merge pull request #644 from miquel-cabanas/v1.6.0-20180814
[IP-724] Add "Invoiced" button in the View Quotes page.

This comment has been minimized.

Copy link

M4rt1n17 commented Jan 11, 2019

/application/third_party/MX/Router.php (line 221)

public function set_class($class)
        $suffix = $this->config->item('controller_suffix');
        if (strpos($class, $suffix) === false) {
            $class .= $suffix;

changed to:

public function set_class($class)
        $suffix = (string) $this->config->item('controller_suffix'); 

Will make it PHP 7.3 compatible.
I think casting $suffix to string is appropriate if controller_suffix is(or can be) empty.
Can someone pls confirm its working?


This comment has been minimized.

Copy link

Kovah commented Jan 30, 2019

Pull requests are not for issues.


@Kovah Kovah closed this Jan 30, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment