DO NOT REPLY [Bug 38243] [patch] gaps in table-columns crash fop

2012-03-31 Thread bugzilla 
https://issues.apache.org/bugzilla/show_bug.cgi?id=38243

Glenn Adams  changed:

   What|Removed |Added

 Status|RESOLVED|CLOSED

--- Comment #4 from Glenn Adams  2012-04-01 06:51:15 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.

Re: Gaps in table columns

2007-10-29 Thread Vincent Hennebert 
Hi Andreas,

Thanks for your inputs.

Andreas L Delmelle wrote:
> On Oct 26, 2007, at 19:35, Andreas L Delmelle wrote:
> 
>> On Oct 26, 2007, at 19:25, Vincent Hennebert wrote:
>>
>>> in layoutmgr.table.ColumnSetup an error message is thrown if there are
>>> gaps between columns. There is also some code in the TableFObj tree to
>>> deal with column gaps. But AFAIU there’s nothing in the spec stating
>>> that fo:table-columns should be consecutive.
>>
>> Indeed not. You could number your columns 100, 200, 300...
>> IIRC, they don't even have to be defined in ascending order, and
>> neither do the table-cells.
>>
>> So, if you'd like, you could define a three column table:
>>
>> 
>>   
>>   
>>   
>>   ...
> 
> It just occurred to me that I wouldn't really know the 'expected' result
> in this case... :-)

A table with 10 columns with potentially border or background 
specifications for columns #1, #5, #10. I’d say.


> I don't believe --could be wrong-- that FOP currently makes any attempt
> to re-order the columns, so column-number 10 will be rendered as the
> first column?

Haven’t figured out yet by looking at the code. But I will fix that if 
necessary.


Cheers,
Vincent

Re: Gaps in table columns

2007-10-26 Thread Andreas L Delmelle 

On Oct 26, 2007, at 19:35, Andreas L Delmelle wrote:


On Oct 26, 2007, at 19:25, Vincent Hennebert wrote:

in layoutmgr.table.ColumnSetup an error message is thrown if there  
are
gaps between columns. There is also some code in the TableFObj  
tree to

deal with column gaps. But AFAIU there’s nothing in the spec stating
that fo:table-columns should be consecutive.


Indeed not. You could number your columns 100, 200, 300...
IIRC, they don't even have to be defined in ascending order, and  
neither do the table-cells.


So, if you'd like, you could define a three column table:


  
  
  
  ...


It just occurred to me that I wouldn't really know the 'expected'  
result in this case... :-)


I don't believe --could be wrong-- that FOP currently makes any  
attempt to re-order the columns, so column-number 10 will be rendered  
as the first column?



Cheers

Andreas

Re: Gaps in table columns

2007-10-26 Thread Andreas L Delmelle 

On Oct 26, 2007, at 19:25, Vincent Hennebert wrote:


in layoutmgr.table.ColumnSetup an error message is thrown if there are
gaps between columns. There is also some code in the TableFObj tree to
deal with column gaps. But AFAIU there’s nothing in the spec stating
that fo:table-columns should be consecutive.


Indeed not. You could number your columns 100, 200, 300...
IIRC, they don't even have to be defined in ascending order, and  
neither do the table-cells.


So, if you'd like, you could define a three column table:


  
  
  
  ...


And as table-columns are
only placeholders for properties that may be used by children
table-cells, I don’t see what is wrong with gaps.


Nothing.



Anyone against me changing this behaviour?


Nope.


Cheers

Andreas

Gaps in table columns

2007-10-26 Thread Vincent Hennebert 
Hi,

in layoutmgr.table.ColumnSetup an error message is thrown if there are 
gaps between columns. There is also some code in the TableFObj tree to 
deal with column gaps. But AFAIU there’s nothing in the spec stating 
that fo:table-columns should be consecutive. And as table-columns are 
only placeholders for properties that may be used by children 
table-cells, I don’t see what is wrong with gaps.

Anyone against me changing this behaviour?

Thanks,
Vincent

DO NOT REPLY [Bug 38243] - [patch] gaps in table-columns crash fop

2006-05-03 Thread bugzilla 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
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=38243


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2006-05-03 08:33 ---
Applied. Thanks. Sorry for the delay. This seems to have slipped through the 
cracks.
http://svn.apache.org/viewcvs?rev=399200&view=rev

-- 
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 38243] - [patch] gaps in table-columns crash fop

2006-01-12 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=38243>.
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=38243





--- Additional Comments From [EMAIL PROTECTED]  2006-01-12 18:47 ---
Created an attachment (id=17400)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=17400&action=view)
Solves problems with gaps in table-columns 


-- 
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 38243] - [patch] gaps in table-columns crash fop

2006-01-12 Thread bugzilla 
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
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=38243





--- Additional Comments From [EMAIL PROTECTED]  2006-01-12 18:47 ---
Created an attachment (id=17401)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=17401&action=view)
example


-- 
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 38243] New: - [patch] gaps in table-columns crash fop

2006-01-12 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=38243>.
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=38243

   Summary: [patch] gaps in table-columns crash fop
   Product: Fop
   Version: 1.0dev
  Platform: Other
OS/Version: other
Status: NEW
  Severity: normal
  Priority: P2
 Component: page-master/layout
AssignedTo: fop-dev@xmlgraphics.apache.org
ReportedBy: [EMAIL PROTECTED]


Gaps in table-columns crash fop because in the current code only the getting of
the column number is skiped and afterwards continued with the _empty_
tablecolumn object before the immeaditly following code is testing for gaps.
This patch should solve the problem.

-- 
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.