-
Notifications
You must be signed in to change notification settings - Fork 31
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
Master table autofill adrm #3783
Conversation
if (row !== selection.bottom) { | ||
this.select(activePosition.col, row); | ||
for (let row = selection.bottom + 1; row <= autofillRow; row++) { | ||
if (this.getters.getEvaluatedCell({ ...activePosition, row }).type !== CellValueType.empty) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm afraid you have to check the core cell because a function can return an empty evaluated cell (think about odoo pivot functions) and you don't want the formula to be overwritten in this case.
This commit improves the `AUTOFILL_AUTO` command for tables. Now when auto filling a cell in a table, we will autofill until the end of the table. Task: 3777825
The autofill done in the `AUTOFILL_AUTO` command wasn't checking if the cells already had a content, and thus could overwrite cells. This commit add a check to stop the autofill at the first non-empty cell encountered. Task: 3777825
b8c5119
to
7df4a94
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
robodoo r+ rebase-ff
Merge method set to rebase and fast-forward. |
This commit improves the `AUTOFILL_AUTO` command for tables. Now when auto filling a cell in a table, we will autofill until the end of the table. Task: 3777825 Part-of: #3783
The autofill done in the `AUTOFILL_AUTO` command wasn't checking if the cells already had a content, and thus could overwrite cells. This commit add a check to stop the autofill at the first non-empty cell encountered. closes #3783 Task: 3777825 Signed-off-by: Lucas Lefèvre (lul) <lul@odoo.com>
Description:
[IMP] autofill:
AUTOFILL_AUTO
for tablesThis commit improves the
AUTOFILL_AUTO
command for tables. Now whenauto filling a cell in a table, we will autofill until the end of the
table.
Task: : 3777825
[FIX] autofill:
AUTOFILL_AUTO
overwrite cellsThe autofill done in the
AUTOFILL_AUTO
command wasn't checking ifthe cells already had a content, and thus could overwrite cells.
This commit add a check to stop the autofill at the first non-empty
cell encountered.
Task: : 3777825
review checklist