Bug 122585

Summary: Classification: by using the dialog, footer text from policy is not placed in the document - it is the same as the header
Product: LibreOffice Reporter: Cor Nouws <cno>
Component: WriterAssignee: Not Assigned <libreoffice-bugs>
Status: RESOLVED WORKSFORME    
Severity: normal CC: aron.budea, cno, ilmari.lauhakangas, jbfaure, raal, vmiklos, xiscofauli
Priority: medium Keywords: regression
Version: 6.0.0.0.alpha0+   
Hardware: All   
OS: All   
See Also: https://bugs.documentfoundation.org/show_bug.cgi?id=122586
Whiteboard:
Crash report or crash signature: Regression By:
Bug Depends on:    
Bug Blocks: 123810    

Description Cor Nouws 2019-01-08 22:08:25 UTC
- Writer document
- View > Toolbar > (TSCP) Classification
- use list from toolbar and apply Confidential
   >  header and footer text appear in the document (as defined in policy file)
     (as well as watermark)
   This is OK

- New document
- Open dialog Document Classification
- select Confidential from list Classification
- hit OK
   > footer texts only shows the "classification", just as the header
     (if in the dialog text is added in the "content" box, that is
      placed in the footer too > header and footer are the same)
     thus the footer also misses the extra info (from the policy file)

It is easy to see per version, by comparing result from toolbar and from dialog!

Already wrong in Version: 6.0.0.0.alpha0+
Build ID: 892c719fffa06de4c7aeab497326cad7bae9e5c6
CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk2; 
TinderBox: Linux-rpm_deb-x86_64@70-TDF, Branch:master, Time: 2017-09-27_03:02:09
Locale: nl-NL (nl_NL.UTF-8); Calc: group

So an implementation error (stupid I missed that before - apologies :\ )
Comment 1 raal 2019-01-09 07:08:58 UTC
Confirm. Version: 6.3.0.0.alpha0+
Build ID: 993913469e74e9db9743004bbe36aa412411671c
CPU threads: 4; OS: Windows 6.1; UI render: default; VCL: win;
Comment 2 Cor Nouws 2019-01-29 11:15:46 UTC
was ok in one the master builds before 6.0.0.0.alpha0+
Comment 3 Aron Budea 2019-03-03 05:57:50 UTC
I don't know much about TSCP Classification, but isn't the point of the dialog that the user constructs the text, and it doesn't come from the policy?

The dialog was added with the following commit, and already doesn't add info from the policy file:
https://cgit.freedesktop.org/libreoffice/core/commit/?id=71ee09947d5a71105d64fd225bb3672dfa7ce834
author		Tomaž Vajngerl <tomaz.vajngerl@collabora.co.uk>	2017-09-19 15:37:04 +0200
committer	Tomaž Vajngerl <quikee@gmail.com>	2017-09-21 09:49:12 +0200

TSCP: add advanced classification dialog
Comment 4 Cor Nouws 2019-03-03 09:22:21 UTC
(In reply to Aron Budea from comment #3)
> I don't know much about TSCP Classification, but isn't the point of the
> dialog that the user constructs the text, and it doesn't come from the
> policy?
It allows the user to add information _additional_ to what the policy provides.
Comment 5 Buovjaga 2020-05-04 16:51:44 UTC
Cor: I tried with Linux 6.0 bibisect repo, but I can't find a build where the footer gets the policy via the dialog. Jumping backwards with 'git checkout HEAD~1000' (or 100), I only arrived at a state, where the toolbar lacks the dialog button and I could not figure out how to launch it (was not implemented yet?).

Can you please investigate this further?
Comment 6 Jean-Baptiste Faure 2022-05-05 15:06:53 UTC
Not reproducible in current master :

Version: 7.4.0.0.alpha0+ / LibreOffice Community
Build ID: 9f1701d01d9f664828356976d8592492f85b30f5
CPU threads: 8; OS: Linux 5.13; UI render: default; VCL: gtk3
Locale: fr-FR (fr_FR.UTF-8); UI: en-US
Ubuntu_20.04_x86-64
Calc: threaded

nor in Version: 7.3.4.0.0+ / LibreOffice Community
Build ID: 614049f7bc962f48676a19b5e6e2ded2862280d6
CPU threads: 8; OS: Linux 5.13; UI render: default; VCL: gtk3
Locale: fr-FR (fr_FR.UTF-8); UI: fr-FR
Ubuntu_20.04_x86-64
Calc: threaded

Closing as WorksForMe. Please feel free to reopen if you disagree.

Best regards. JBF