[Excel2007+] Pre-defined pictures are losing the move and size properties about abap2xlsx HOT 1 OPEN

Thanks for notifying about this issue, and for this very detailed analysis.

Reproduced. it works fine for oneCellAnchor which seems to always correspond to "move but don't size", but fails for 2 out of 3 cases with twoCellAnchor, it's always considered as "move and size". The attribute editAs is currently not considered by abap2xlsx.

I did the test on these four files with the demo program zdemo_excel37 which duplicates a file, I did the test with zcl_excel_reader_2007 and zcl_excel_writer_2007:

• issue-1180-drawing-resize-oneCellAnchor-move-but-dont-size.xlsx
• issue-1180-drawing-resize-twoCellAnchor-move-but-dont-size.xlsx
• issue-1180-drawing-resize-twoCellAnchor-move-and-size.xlsx
• issue-1180-drawing-resize-twoCellAnchor-dont-move-or-size.xlsx

Concerned code:

• abap2xlsx/src/zcl_excel_reader_2007.clas.abap

  Lines 3624 to 3654 in 5f784c8

  	drawing = me->get_ixml_from_zip_archive( ip_path ).
  	* one-cell anchor **************
  	anchors = drawing->get_elements_by_tag_name_ns( name = 'oneCellAnchor' uri = namespace-xdr ).
  	coll_length = anchors->get_length( ).
  	iterator = anchors->create_iterator( ).
  	DO coll_length TIMES.
  	anchor_elem ?= iterator->get_next( ).
  	CALL METHOD me->load_drawing_anchor
  	EXPORTING
  	io_anchor_element = anchor_elem
  	io_worksheet = io_worksheet
  	it_related_drawings = rel_drawings.
  	ENDDO.
  	* two-cell anchor ******************
  	anchors = drawing->get_elements_by_tag_name_ns( name = 'twoCellAnchor' uri = namespace-xdr ).
  	coll_length = anchors->get_length( ).
  	iterator = anchors->create_iterator( ).
  	DO coll_length TIMES.
  	anchor_elem ?= iterator->get_next( ).
  	CALL METHOD me->load_drawing_anchor
  	EXPORTING
  	io_anchor_element = anchor_elem
  	io_worksheet = io_worksheet
  	it_related_drawings = rel_drawings.
  	ENDDO.

• abap2xlsx/src/zcl_excel_writer_2007.clas.abap

  Lines 2833 to 2927 in 5f784c8

  	ls_position = io_drawing->get_position( ).
  	IF ls_position-anchor = 'ONE'.
  	ep_anchor = io_document->create_simple_element( name = lc_xml_node_onecellanchor
  	parent = io_document ).
  	ELSE.
  	ep_anchor = io_document->create_simple_element( name = lc_xml_node_twocellanchor
  	parent = io_document ).
  	ENDIF.
  	* from cell ******************************
  	lo_element_from = io_document->create_simple_element( name = lc_xml_node_from
  	parent = io_document ).
  	lv_col = ls_position-from-col.
  	lv_row = ls_position-from-row.
  	lv_col_offset = ls_position-from-col_offset.
  	lv_row_offset = ls_position-from-row_offset.
  	CONDENSE lv_col NO-GAPS.
  	CONDENSE lv_row NO-GAPS.
  	CONDENSE lv_col_offset NO-GAPS.
  	CONDENSE lv_row_offset NO-GAPS.
  	lo_element = io_document->create_simple_element( name = lc_xml_node_col
  	parent = io_document ).
  	lo_element->set_value( value = lv_col ).
  	lo_element_from->append_child( new_child = lo_element ).
  	lo_element = io_document->create_simple_element( name = lc_xml_node_coloff
  	parent = io_document ).
  	lo_element->set_value( value = lv_col_offset ).
  	lo_element_from->append_child( new_child = lo_element ).
  	lo_element = io_document->create_simple_element( name = lc_xml_node_row
  	parent = io_document ).
  	lo_element->set_value( value = lv_row ).
  	lo_element_from->append_child( new_child = lo_element ).
  	lo_element = io_document->create_simple_element( name = lc_xml_node_rowoff
  	parent = io_document ).
  	lo_element->set_value( value = lv_row_offset ).
  	lo_element_from->append_child( new_child = lo_element ).
  	ep_anchor->append_child( new_child = lo_element_from ).
  	IF ls_position-anchor = 'ONE'.
  	* ext ******************************
  	lo_element_ext = io_document->create_simple_element( name = lc_xml_node_ext
  	parent = io_document ).
  	lv_value = io_drawing->get_width_emu_str( ).
  	lo_element_ext->set_attribute_ns( name = 'cx'
  	value = lv_value ).
  	lv_value = io_drawing->get_height_emu_str( ).
  	lo_element_ext->set_attribute_ns( name = 'cy'
  	value = lv_value ).
  	ep_anchor->append_child( new_child = lo_element_ext ).
  	ELSEIF ls_position-anchor = 'TWO'.
  	* to cell ******************************
  	lo_element_to = io_document->create_simple_element( name = lc_xml_node_to
  	parent = io_document ).
  	lv_col = ls_position-to-col.
  	lv_row = ls_position-to-row.
  	lv_col_offset = ls_position-to-col_offset.
  	lv_row_offset = ls_position-to-row_offset.
  	CONDENSE lv_col NO-GAPS.
  	CONDENSE lv_row NO-GAPS.
  	CONDENSE lv_col_offset NO-GAPS.
  	CONDENSE lv_row_offset NO-GAPS.
  	lo_element = io_document->create_simple_element( name = lc_xml_node_col
  	parent = io_document ).
  	lo_element->set_value( value = lv_col ).
  	lo_element_to->append_child( new_child = lo_element ).
  	lo_element = io_document->create_simple_element( name = lc_xml_node_coloff
  	parent = io_document ).
  	lo_element->set_value( value = lv_col_offset ).
  	lo_element_to->append_child( new_child = lo_element ).
  	lo_element = io_document->create_simple_element( name = lc_xml_node_row
  	parent = io_document ).
  	lo_element->set_value( value = lv_row ).
  	lo_element_to->append_child( new_child = lo_element ).
  	lo_element = io_document->create_simple_element( name = lc_xml_node_rowoff
  	parent = io_document ).
  	lo_element->set_value( value = lv_row_offset ).
  	lo_element_to->append_child( new_child = lo_element ).
  	ep_anchor->append_child( new_child = lo_element_to ).
  	ENDIF.

from abap2xlsx.