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

NEW Do not include cancelled orders in the billable filter #31119

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

zephyriony
Copy link
Contributor

NEW Do not include cancelled orders in the billable filter

@@ -941,6 +941,9 @@
}
if ($search_billed != '' && $search_billed >= 0) {
$sql .= ' AND c.facture = '.((int) $search_billed);
if (getDolGlobalString('HIDE_CANCELLED_ORDERS_BILLABLE') == 1) {
Copy link
Member

@eldy eldy Sep 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the user need both a filter on
billed
+
status of order that is not canceled.

The user can set the two filters to match its need. Why adding an option to replace silently a filter that should be already available to end user ?

Copy link
Contributor Author

@zephyriony zephyriony Sep 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logically, by definition, a cancelled order is not billable.
I had the same reaction as you when this remark was made to me in my company, but the secretary told me that it was a pain to have to select both the invoiced and the three statuses.
Especially as Dolibarr gives the wrong information, for me it's almost a bug fix, it's cancelled and can't be billed.
I've left the constant to let the user choose, but I think they should be hidden by default.
For example, our Dolibarr gives us €10,000 to invoice, but the reality is that we have a lot less to invoice and €10,000 in cancelled orders. The idea of the billable status is to see the backlog and be able to bill what I think is really billable.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having a filter that does not match the filter defined by user is a bug.
If user need to select 2 filters,this is an interface problem. And this must be fixed by interface changes.
Can you provide a screenshot of the page or menu where the secretary click before having to modify the filter ?

Copy link
Contributor Author

@zephyriony zephyriony Sep 26, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
image

There are other orders that I have removed for reasons of confidentiality.

Currently, as you suggested, you can use the “Status” filter, but it's a pain to select all three open statuses.
This seems to me to be a bug, since I think that in the majority of companies, a cancelled order is not billable.
The total amount is therefore not correct, the two cancelled orders need to be removed. Indeed, as you say, we could filter by status, but that would take longer and be illogical, I insist, there's a lack of consistency, in most cases we don't invoice a cancelled order.
The constant leaves the choice to those who want to bill them

Copy link
Member

@eldy eldy Sep 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the filter "Billed" is already set correctly.
To reach this page, where does he/she click ? From the entry "Billlable orders" in the left menu ?
I check with last version of v19, v20 en develop and the filter "Status" that exclude draft order is correctly set automatically.
What is the version used ?

@eldy eldy added the Discussion Some questions or discussions are opened and wait answers of author or other people to be processed label Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Some questions or discussions are opened and wait answers of author or other people to be processed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants