censor/Censor
1
37
Fork
You've already forked Censor
3

bug?: saving redacted files ,by overwriting the original,does not work #90

Closed
opened 2026年02月26日 21:00:58 +01:00 by martin762green · 8 comments
Contributor
Copy link

Assuming this is a different issue to the #78 previously raised.
I am getting consistent failure to "save As" a redacted pdf file by "save as" attempting to overwrite the original file name. I am seeing the error message
ValueError("save to original must be incremental")

Seen on Ubuntu 25.10 with both aarch64 and x86_64 hardware

cheers Martin

Assuming this is a different issue to the #78 previously raised. I am getting consistent failure to "save As" a redacted pdf file by "save as" attempting to overwrite the original file name. I am seeing the error message **ValueError("save to original must be incremental")** Seen on Ubuntu 25.10 with both aarch64 and x86_64 hardware cheers Martin
Author
Contributor
Copy link

Should have added this behaviour is true for the stable down load from flathub (0.3) and the latest main repository

Should have added this behaviour is true for the stable down load from flathub (0.3) and the latest main repository
Author
Contributor
Copy link

Having now looked at the pymupdf documents and the meaning of "incremental" in this context this may be by design rather than a bug. I wqill close when confirmed.
cheers

Having now looked at the pymupdf documents and the meaning of "incremental" in this context this may be by design rather than a bug. I wqill close when confirmed. cheers

Thanks for reporting this issue, and it is quite possible that it is same as #78, but I wasn’t able to replicate till now. And nice that you gave the important hint about the ValueError. Can you confirm that you tried saving a redacted document which includes signatures?

I could reproduce the issue, e.g., with the first test file from here: https://www.tecxoft.com/samples.php. That is unexpected and should be fixed within Censor.

Thanks for reporting this issue, and it is quite possible that it is same as #78, but I wasn’t able to replicate till now. And nice that you gave the important hint about the ValueError. Can you confirm that you tried saving a redacted document which includes signatures? I could reproduce the issue, e.g., with the first test file from here: https://www.tecxoft.com/samples.php. That is unexpected and should be fixed within Censor.
mahlzahn added this to the v0.4.0 milestone 2026年02月27日 12:41:25 +01:00
Author
Contributor
Copy link

My original testing was on a scanned random document. Not sure what you meant by "signature" so I have done a quick check on the stable v0.3
1 added a freeform signature and re scanned as pdf.
2 Added an electronic signature as text
3 added an electronic signature as free-forrm drawing

In each case redacting the document I "saved as "successfully with a new file name.
Unsuccessful trying to overwrite the existing file name. Value error same as before.

Are you sure this is not down to pymupdf by design?
Cheers

My original testing was on a scanned random document. Not sure what you meant by "signature" so I have done a quick check on the stable v0.3 1 added a freeform signature and re scanned as pdf. 2 Added an electronic signature as text 3 added an electronic signature as free-forrm drawing In each case redacting the document I "saved as "successfully with a new file name. Unsuccessful trying to overwrite the existing file name. Value error same as before. Are you sure this is not down to pymupdf by design? Cheers
Author
Contributor
Copy link

PS also used tecxoft snippet and same results
Cheers

PS also used tecxoft snippet and same results Cheers

@martin762green wrote in #90 (comment):

My original testing was on a scanned random document.

True, indeed it is independent of signature and happens to other documents, too.

@martin762green wrote in https://codeberg.org/censor/Censor/issues/90#issuecomment-11050373: > My original testing was on a scanned random document. True, indeed it is independent of signature and happens to other documents, too.

May I ask you to test the flatpak (x86_64) from #92, please? You can download and install it with:

curl https://gitlab.gnome.org/mahlzahn/censor-ci/-/jobs/6212525/artifacts/raw/censor-dev.flatpak -o censor-dev.flatpak
flatpak install --user censor-dev.flatpak
flatpak run --user page.codeberg.censor.Censor.Devel
May I ask you to test the flatpak (x86_64) from #92, please? You can download and install it with: ``` curl https://gitlab.gnome.org/mahlzahn/censor-ci/-/jobs/6212525/artifacts/raw/censor-dev.flatpak -o censor-dev.flatpak flatpak install --user censor-dev.flatpak flatpak run --user page.codeberg.censor.Censor.Devel ```
Author
Contributor
Copy link

@mahlzahn: As requested above. Job done! see #92

@mahlzahn: As requested above. Job done! see #92
Sign in to join this conversation.
No Branch/Tag specified
main
v0.7.3
v0.7.2
v0.7.1
v0.7.0
v0.6.0
v0.5.0
v0.4.0
v0.3.0
v0.2.0
v0.1.0
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
censor/Censor#90
Reference in a new issue
censor/Censor
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?