Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-09 Thread Jean-Marc Lasgouttes
José == José Matos [EMAIL PROTECTED] writes: All of this make perfectly sense to me. Although Jean-Marc has left us with some FUD (but without a given proof), I would like to commit his patch. Could someone please give another OK? José I would like to wait for Jean-Marc on this. Not that I

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-09 Thread Michael Gerz
Jean-Marc Lasgouttes schrieb: I did not manage to reproduce the problem I saw and anyway I do not think they are introduced by the patch. Please go ahead. Committed as http://www.lyx.org/trac/changeset/19019 Michael

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-09 Thread Jean-Marc Lasgouttes
Michael == Michael Gerz [EMAIL PROTECTED] writes: Michael Jean-Marc Lasgouttes schrieb: I did not manage to reproduce the problem I saw and anyway I do not think they are introduced by the patch. Please go ahead. Committed as Michael http://www.lyx.org/trac/changeset/19019 Thanks. JMarc

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-09 Thread Jean-Marc Lasgouttes
> "José" == José Matos <[EMAIL PROTECTED]> writes: >> All of this make perfectly sense to me. Although Jean-Marc has left >> us with some FUD (but without a given proof), I would like to >> commit his patch. Could someone please give another OK? José> I would like to wait for Jean-Marc on

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-09 Thread Michael Gerz
Jean-Marc Lasgouttes schrieb: I did not manage to reproduce the problem I saw and anyway I do not think they are introduced by the patch. Please go ahead. Committed as http://www.lyx.org/trac/changeset/19019 Michael

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-09 Thread Jean-Marc Lasgouttes
> "Michael" == Michael Gerz <[EMAIL PROTECTED]> writes: Michael> Jean-Marc Lasgouttes schrieb: >> I did not manage to reproduce the problem I saw and anyway I do not >> think they are introduced by the patch. Please go ahead. >> Committed as Michael> http://www.lyx.org/trac/changeset/19019

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-07 Thread Michael Gerz
Jean-Marc Lasgouttes schrieb: Jean-Marc Something like that. I tested briefly with 3 different Jean-Marc authors, and I think something is wrong (I think it was Jean-Marc already wrong before, actually). I do not see how current Jean-Marc author could always be index 0 (which is written in the

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-07 Thread José Matos
On Saturday 07 July 2007 12:34:35 Michael Gerz wrote: Accept all changes by Mike and MG:   \tracking_changes true   \output_changes false   \author   \author   \author Michael Gerz [EMAIL PROTECTED] Is there any reason to leave the empty authors field there? All of this make

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-07 Thread Michael Gerz
José Matos schrieb: On Saturday 07 July 2007 12:34:35 Michael Gerz wrote: Accept all changes by Mike and MG: \tracking_changes true \output_changes false \author \author \author Michael Gerz [EMAIL PROTECTED] Is there any reason to leave the empty authors field there?

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-07 Thread Michael Gerz
Jean-Marc Lasgouttes schrieb: Jean-Marc> Something like that. I tested briefly with 3 different Jean-Marc> authors, and I think something is wrong (I think it was Jean-Marc> already wrong before, actually). I do not see how current Jean-Marc> author could always be index 0 (which is written in

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-07 Thread José Matos
On Saturday 07 July 2007 12:34:35 Michael Gerz wrote: > Accept all changes by "Mike" and "MG": > >   \tracking_changes true >   \output_changes false >   \author "" >   \author "" >   \author "Michael Gerz" [EMAIL PROTECTED] Is there any reason to leave the empty authors field there? > All of

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-07 Thread Michael Gerz
José Matos schrieb: On Saturday 07 July 2007 12:34:35 Michael Gerz wrote: Accept all changes by "Mike" and "MG": \tracking_changes true \output_changes false \author "" \author "" \author "Michael Gerz" [EMAIL PROTECTED] Is there any reason to leave the empty authors

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-05 Thread Michael Gerz
Jean-Marc, Jean-Marc Something like that. Great. Very compact and non-intrusive patch! Jean-Marc I tested briefly with 3 different Jean-Marc authors, and I think something is wrong (I think it was Jean-Marc already wrong before, actually). I do not see how current Jean-Marc author could

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-05 Thread Michael Gerz
Jean-Marc, Jean-Marc> Something like that. Great. Very compact and non-intrusive patch! Jean-Marc> I tested briefly with 3 different Jean-Marc> authors, and I think something is wrong (I think it was Jean-Marc> already wrong before, actually). I do not see how current Jean-Marc> author could

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-04 Thread Jean-Marc Lasgouttes
Michael == Michael Gerz [EMAIL PROTECTED] writes: Michael Jean-Marc Lasgouttes schrieb: http://bugzilla.lyx.org/show_bug.cgi?id=3764 I do not like this patch very much, actually %-| The name AuthorActive is ugly, the class definition in Author.h is weird (but better IMO than including map

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-04 Thread Jean-Marc Lasgouttes
Jean-Marc == Jean-Marc Lasgouttes [EMAIL PROTECTED] writes: Jean-Marc A const method should return a const object. If you want to Jean-Marc go this route, the best may be to declare 'used_' as Jean-Marc mutable. Something like that. I tested briefly with 3 different authors, and I think

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-04 Thread Jean-Marc Lasgouttes
Jean-Marc == Jean-Marc Lasgouttes [EMAIL PROTECTED] writes: Jean-Marc == Jean-Marc Lasgouttes [EMAIL PROTECTED] writes: Jean-Marc A const method should return a const object. If you want to Jean-Marc go this route, the best may be to declare 'used_' as Jean-Marc mutable. Jean-Marc Something

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-04 Thread Jean-Marc Lasgouttes
> "Michael" == Michael Gerz <[EMAIL PROTECTED]> writes: Michael> Jean-Marc Lasgouttes schrieb: >> http://bugzilla.lyx.org/show_bug.cgi?id=3764 >> >> I do not like this patch very much, actually %-| The name >> AuthorActive is ugly, the class definition in Author.h is weird >> (but better IMO

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-04 Thread Jean-Marc Lasgouttes
> "Jean-Marc" == Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: Jean-Marc> A const method should return a const object. If you want to Jean-Marc> go this route, the best may be to declare 'used_' as Jean-Marc> mutable. Something like that. I tested briefly with 3 different authors, and I

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-04 Thread Jean-Marc Lasgouttes
> "Jean-Marc" == Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: > "Jean-Marc" == Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes: Jean-Marc> A const method should return a const object. If you want to Jean-Marc> go this route, the best may be to declare 'used_' as Jean-Marc> mutable.

[PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-03 Thread Jean-Marc Lasgouttes
http://bugzilla.lyx.org/show_bug.cgi?id=3764 This is an important privacy bug, that should be fixed before 1..50 IMO. In the following patch, I tried to be as unintrusive as possible. The idea is that before writing the buffer params, the whole document is scanned to determine what are the

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-03 Thread Michael Gerz
Jean-Marc Lasgouttes schrieb: http://bugzilla.lyx.org/show_bug.cgi?id=3764 This is an important privacy bug, that should be fixed before 1..50 IMO. In the following patch, I tried to be as unintrusive as possible. Jean-Marc, thank you very much for stepping forward! The other authors of

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-03 Thread Jean-Marc Lasgouttes
Michael == Michael Gerz [EMAIL PROTECTED] writes: Michael Jean-Marc, thank you very much for stepping forward! I had the impression that nobody was going to do it :) Michael A consequence of this patch is that the list of authors grows Michael with every load-buffer-save-buffer cycle. Luckily,

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-03 Thread Michael Gerz
Jean-Marc Lasgouttes schrieb: What do you mean? That it is saved in the file with id 0 for all his changes? Then I do not understand what happens when somebody else edits the file... This is precisely an area I do not understand. The active user (i.e. the one running LyX) always has author id

[PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-03 Thread Jean-Marc Lasgouttes
http://bugzilla.lyx.org/show_bug.cgi?id=3764 This is an important privacy bug, that should be fixed before 1..50 IMO. In the following patch, I tried to be as unintrusive as possible. The idea is that before writing the buffer params, the whole document is scanned to determine what are the

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-03 Thread Michael Gerz
Jean-Marc Lasgouttes schrieb: http://bugzilla.lyx.org/show_bug.cgi?id=3764 This is an important privacy bug, that should be fixed before 1..50 IMO. In the following patch, I tried to be as unintrusive as possible. Jean-Marc, thank you very much for stepping forward! The other authors of

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-03 Thread Jean-Marc Lasgouttes
> "Michael" == Michael Gerz <[EMAIL PROTECTED]> writes: Michael> Jean-Marc, thank you very much for stepping forward! I had the impression that nobody was going to do it :) Michael> A consequence of this patch is that the list of authors grows Michael> with every load-buffer-save-buffer

Re: [PATCH] Bug 3764: Implicit \author field in .lyx files

2007-07-03 Thread Michael Gerz
Jean-Marc Lasgouttes schrieb: What do you mean? That it is saved in the file with id 0 for all his changes? Then I do not understand what happens when somebody else edits the file... This is precisely an area I do not understand. The active user (i.e. the one running LyX) always has author id

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-20 Thread Georg Baum
Michael Gerz wrote: Pavel Sanda schrieb: In contrary, my resources are limited at the moment. the quick fix is below. I think this is a good work-around for the time being (i.e. 1.5rc2). I don't think so, IMHO this hack causes more harm than good: It affects only new users, not

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-20 Thread Georg Baum
Michael Gerz wrote: > Pavel Sanda schrieb: >>> In contrary, my resources are limited at the moment. >>> >> >> the quick fix is below. >> > I think this is a good work-around for the time being (i.e. 1.5rc2). I don't think so, IMHO this hack causes more harm than good: It affects only new

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-19 Thread Pavel Sanda
In contrary, my resources are limited at the moment. the quick fix is below. Setting the user name to anonymous may be a good solution for the time being (at least for 1.5rc2) provided that we can change back to the old settings at no cost later. afaik the current situation is : 1. user

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-19 Thread Michael Gerz
Pavel Sanda schrieb: In contrary, my resources are limited at the moment. the quick fix is below. I think this is a good work-around for the time being (i.e. 1.5rc2). If I get another OK, I will commit. As soon as we have a better solution, I am going to revert the quick fix.

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-19 Thread José Matos
On Tuesday 19 June 2007 21:58:16 Michael Gerz wrote: If I get another OK, I will commit. As soon as we have a better solution, I am going to revert the quick fix. Michael OK. -- José Abílio

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-19 Thread Pavel Sanda
> In contrary, my resources are limited at the moment. the quick fix is below. > Setting the user name to anonymous may be a good solution for the time being > (at least for 1.5rc2) provided that we can change back to the old settings at > no cost later. afaik the current situation is : 1.

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-19 Thread Michael Gerz
Pavel Sanda schrieb: In contrary, my resources are limited at the moment. the quick fix is below. I think this is a good work-around for the time being (i.e. 1.5rc2). If I get another OK, I will commit. As soon as we have a better solution, I am going to revert the quick fix.

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-19 Thread José Matos
On Tuesday 19 June 2007 21:58:16 Michael Gerz wrote: > If I get another OK, I will commit. As soon as we have a better > solution, I am going to revert the quick fix. > > Michael OK. -- José Abílio

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-18 Thread Pavel Sanda
Any comments or suggestions for a better solution? Michael, what if we just change the code inside userinfo.cpp instead of intricating CT code ? the solution could be that the user_name/mail() will return eg Anonymous by default. later (1.5.1 or so) - lyx initialization could be supplied by

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-18 Thread michael . gerz
Michael, what if we just change the code inside userinfo.cpp instead of intricating CT code ? the solution could be that the user_name/mail() will return eg Anonymous by default. I have some ideas on how to fix the problem without changing the file format. In any case (with or without

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-18 Thread Pavel Sanda
> > Any comments or suggestions for a better solution? Michael, what if we just change the code inside userinfo.cpp instead of intricating CT code ? the solution could be that the user_name/mail() will return eg "Anonymous" by default. later (1.5.1 or so) - lyx initialization could be supplied

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-18 Thread michael . gerz
> Michael, what if we just change the code inside userinfo.cpp instead of > intricating CT code ? > the solution could be that the user_name/mail() will return eg > "Anonymous" by default. I have some ideas on how to fix the problem without changing the file format. In any case (with or

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-03 Thread Michael Gerz
Jean-Marc Lasgouttes schrieb: Michael In the LyX document header, we have to list all authors for Michael which a change is tracked in the document. That means that we Michael have to iterate through the complete document and check Michael whether there is any change for a given author whenever

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-03 Thread Bernhard Roider
Michael Gerz schrieb: Jean-Marc Lasgouttes schrieb: Michael In the LyX document header, we have to list all authors for Michael which a change is tracked in the document. That means that we Michael have to iterate through the complete document and check Michael whether there is any change for a

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-03 Thread Pavel Sanda
Question: Would it be possible to change the file format at this stage of development? E.g., to i remember jose has written somewhere that yes. fileformat cant be changed only after 1.5.0 is out to maintain 1.5.x series stable. Any comments or suggestions for a better solution? just two

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-03 Thread Michael Gerz
Jean-Marc Lasgouttes schrieb: Michael> In the LyX document header, we have to list all authors for Michael> which a change is tracked in the document. That means that we Michael> have to iterate through the complete document and check Michael> whether there is any change for a given author

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-03 Thread Bernhard Roider
Michael Gerz schrieb: Jean-Marc Lasgouttes schrieb: Michael> In the LyX document header, we have to list all authors for Michael> which a change is tracked in the document. That means that we Michael> have to iterate through the complete document and check Michael> whether there is any change

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-03 Thread Pavel Sanda
> Question: Would it be possible to change the file format at this stage > of development? E.g., to i remember jose has written somewhere that yes. fileformat cant be changed only after 1.5.0 is out to maintain 1.5.x series stable. > Any comments or suggestions for a better solution? just two

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-01 Thread Michael Gerz
Michael Gerz schrieb: Juergen Spitzmueller schrieb: Michael Gerz wrote: You can have changes in a document even if change tracking is deactivated! (We no longer have the enervating restriction of the 1.4.X series) But the possibility to hide author information is a must. This is a

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-01 Thread Jean-Marc Lasgouttes
Michael == Michael Gerz [EMAIL PROTECTED] writes: Michael In the LyX document header, we have to list all authors for Michael which a change is tracked in the document. That means that we Michael have to iterate through the complete document and check Michael whether there is any change for a

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-01 Thread Michael Gerz
Michael Gerz schrieb: Juergen Spitzmueller schrieb: Michael Gerz wrote: You can have changes in a document even if change tracking is deactivated! (We no longer have the enervating restriction of the 1.4.X series) But the possibility to hide author information is a must. This is a

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-06-01 Thread Jean-Marc Lasgouttes
> "Michael" == Michael Gerz <[EMAIL PROTECTED]> writes: Michael> In the LyX document header, we have to list all authors for Michael> which a change is tracked in the document. That means that we Michael> have to iterate through the complete document and check Michael> whether there is any

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-05-30 Thread Michael Gerz
Veto! You can have changes in a document even if change tracking is deactivated! (We no longer have the enervating restriction of the 1.4.X series) Michael Pavel Sanda schrieb: Hi, the following patch add author info to .lyx file only if change tracking is enabled (

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-05-30 Thread Juergen Spitzmueller
Michael Gerz wrote: You can have changes in a document even if change tracking is deactivated! (We no longer have the enervating restriction of the 1.4.X series) But the possibility to hide author information is a must. This is a serious security hole. Jürgen

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-05-30 Thread Michael Gerz
Juergen Spitzmueller schrieb: Michael Gerz wrote: You can have changes in a document even if change tracking is deactivated! (We no longer have the enervating restriction of the 1.4.X series) But the possibility to hide author information is a must. This is a serious security hole.

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-05-30 Thread Lars Gullik Bjønnes
Michael Gerz [EMAIL PROTECTED] writes: | Juergen Spitzmueller schrieb: | Michael Gerz wrote: | | | You can have changes in a document even if change tracking is deactivated! | | (We no longer have the enervating restriction of the 1.4.X series) | | | But the possibility to hide author

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-05-30 Thread Michael Gerz
Veto! You can have changes in a document even if change tracking is deactivated! (We no longer have the enervating restriction of the 1.4.X series) Michael Pavel Sanda schrieb: Hi, the following patch add author info to .lyx file only if change tracking is enabled (

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-05-30 Thread Juergen Spitzmueller
Michael Gerz wrote: > You can have changes in a document even if change tracking is deactivated! > > (We no longer have the enervating restriction of the 1.4.X series) But the possibility to hide author information is a must. This is a serious security hole. Jürgen

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-05-30 Thread Michael Gerz
Juergen Spitzmueller schrieb: Michael Gerz wrote: You can have changes in a document even if change tracking is deactivated! (We no longer have the enervating restriction of the 1.4.X series) But the possibility to hide author information is a must. This is a serious security hole.

Re: [patch] bug 3764: Implicit \author field in .lyx files

2007-05-30 Thread Lars Gullik Bjønnes
Michael Gerz <[EMAIL PROTECTED]> writes: | Juergen Spitzmueller schrieb: | > Michael Gerz wrote: | > | > | >> You can have changes in a document even if change tracking is deactivated! | >> | >> (We no longer have the enervating restriction of the 1.4.X series) | >> | > | > But the possibility to

[patch] bug 3764: Implicit \author field in .lyx files

2007-05-29 Thread Pavel Sanda
Hi, the following patch add author info to .lyx file only if change tracking is enabled ( http://bugzilla.lyx.org/show_bug.cgi?id=3764 ). Pavel --- work/lyx-1.5.0beta3/src/BufferParams.cpp2007-05-12 02:22:37.0 +0200 +++ /trash/lyx-1.5.0beta3/src/BufferParams.cpp 2007-05-30

[patch] bug 3764: Implicit \author field in .lyx files

2007-05-29 Thread Pavel Sanda
Hi, the following patch add author info to .lyx file only if change tracking is enabled ( http://bugzilla.lyx.org/show_bug.cgi?id=3764 ). Pavel --- work/lyx-1.5.0beta3/src/BufferParams.cpp2007-05-12 02:22:37.0 +0200 +++ /trash/lyx-1.5.0beta3/src/BufferParams.cpp 2007-05-30