DO NOT REPLY [Bug 38087] [patch] force-page-count property implementation

2012-04-01 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=38087

Glenn Adams gl...@skynav.com changed:

   What|Removed |Added

 Status|RESOLVED|CLOSED

--- Comment #12 from Glenn Adams gl...@skynav.com 2012-04-01 06:39:38 UTC ---
batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.


DO NOT REPLY [Bug 38087] - [patch] force-page-count property implementation

2006-01-10 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38087.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38087


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2006-01-10 21:21 ---
Committed the test cases. Thanks. Simon

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.


DO NOT REPLY [Bug 38087] - [patch] force-page-count property implementation

2006-01-09 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38087.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38087


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|RESOLVED|REOPENED
 Resolution|FIXED   |




--- Additional Comments From [EMAIL PROTECTED]  2006-01-09 22:20 ---
Thanks for the fine test cases. Re noauto: Do you really expect 18 pages? I get
17, and I think that is correct.


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.


DO NOT REPLY [Bug 38087] - [patch] force-page-count property implementation

2006-01-09 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38087.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38087





--- Additional Comments From [EMAIL PROTECTED]  2006-01-10 00:38 ---
Yes i expect 17. I want to show the missing pageno 217 - a typical case of
self-delusion. 

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.


DO NOT REPLY [Bug 38087] - [patch] force-page-count property implementation

2006-01-06 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38087.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38087


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2006-01-06 22:33 ---
Applied. I rearranged the calls somewhat. This is the resulting call hierarchy:

PageSequence.startOfNode
 AreaTreeHandler.startPageSequence
  PSLM.doForcePageCount (previous PS if not null)
   PSLM.finishPage
  PSLM.finishPageSequence (previous PS if not null)
   Root.notifyPageSequenceFinished
   AreaTreeHandler.notifyPageSequenceFinished
  PageSequence.initPageNumber

PageSequence.endOfNode
 AreaTreeHandler.endPageSequence
  PSLM.activateLayout
   PSLM.finishPage

Thanks for the patch.

Simon


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.


Re: DO NOT REPLY [Bug 38087] - [patch] force-page-count property implementation

2006-01-04 Thread Simon Pepping
On Wed, Jan 04, 2006 at 02:08:56PM +0100, gerhard oettl wrote:
 --- Additional Comments From [EMAIL PROTECTED]  2006-01-03 22:23 ---
 I am not quite happy with the fact that checkForcePageCount is called
 by the next page sequence. This is an interaction between page
 sequences, and it is better dealt with by the controlling structure,
 in this case AreaTreeHandler. In other words, I go with your
 suggestion in the second *.
 
 I had better orderd the open questions - so think them numbered:
 - 1) 
 - 2)
   * 2a) 
   * 2b) 
   * 2c)
 
 ad 1) 
 If there is no precedence for one of the choices i will change
 previousPageSeqLM from public to private as proposed in the question.

Agreed.
 
 ad 2) 
 I think it is the second - what you call the second * ?
 Because i couldn't think it a good idea to preserve the 
 PageSequenceLayoutManger at layoutmanger level and call it from area 
 level, so i think i shall do the changes 2a, 2b and 2c alltogether.
 2c is a must anyway.

2b is the second *. I agree with your proposal to do all three changes
2a-c.

 There is a little error in the last page sequence but one in your demo
 file. It says that next is auto-even, which is not true.
 
 yes
 If of any importance i can add a corrected fo to show the possibility 
 of missing pagenumbers in case of force-page-count=no-force.
 Let me know.

It would be good if you could turn it into a test file for the
layoutengine tests, which highlights all relevant
combinations of force-page-count and initial-page-number. Otherwise I
will do it.

  .''`.   gerhard oettl   on   Debian/Gnu Linux
That's the best, as soon as I find out how to make my mike work. :-)

Regards, Simon

-- 
Simon Pepping
home page: http://www.leverkruid.nl



DO NOT REPLY [Bug 38087] - [patch] force-page-count property implementation

2006-01-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=38087.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=38087


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Attachment #17303|0   |1
is obsolete||




--- Additional Comments From [EMAIL PROTECTED]  2006-01-04 23:59 ---
Created an attachment (id=17329)
 -- (http://issues.apache.org/bugzilla/attachment.cgi?id=17329action=view)
patch 2 (using area tree)

ad the now 2c)
At the time the checkForcePageCount is called from the
AreaTreeHandler the initialPageNumber is already calced by the
PageSequence without knowing about eventueally later added pages so 
it must be recalced. I did this by making the initPageNumber method
public. If this should stay private i have to add a public
recalcInitPageNumber method that does nothing than call the
private initPageNumber method - at your decision.


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.