Merged
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Codecov Report❌ Patch coverage is 📢 Thoughts on this report? Let us know! |
Add summary and text report steps to show detailed information when PHPMD finds issues, instead of just directing to Security tab. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Split documentate_get_resolucion_demo_data() into 3 helper functions to comply with PHPMD ExcessiveMethodLength rule (max 150 lines). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Comment on lines
2071
to
2192
| @@ -2001,9 +2118,10 @@ private static function convert_table_node_to_docx( DOMDocument $doc, DOMElement | |||
| * @param DOMElement|null $base_rpr Base run properties. | |||
| * @param array<string,bool> $formatting Formatting flags. | |||
| * @param array<string,mixed>|null $relationships Relationships context. | |||
| * @param string|null $alignment Cell alignment (left, center, right, justify). | |||
| * @return array<int,DOMElement> Array of paragraph elements. | |||
| */ | |||
| private static function convert_cell_content_to_docx( DOMDocument $doc, $children, $base_rpr, array $formatting, &$relationships ) { | |||
| private static function convert_cell_content_to_docx( DOMDocument $doc, $children, $base_rpr, array $formatting, &$relationships, $alignment = null ) { | |||
| $result = array(); | |||
| $current_runs = array(); | |||
|
|
|||
| @@ -2027,7 +2145,7 @@ private static function convert_cell_content_to_docx( DOMDocument $doc, $childre | |||
| if ( 'ul' === $tag || 'ol' === $tag ) { | |||
| // Flush any accumulated inline runs. | |||
| if ( ! empty( $current_runs ) ) { | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr ); | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr, $alignment ); | |||
| $current_runs = array(); | |||
| } | |||
| // Convert list to paragraphs. | |||
| @@ -2039,7 +2157,7 @@ private static function convert_cell_content_to_docx( DOMDocument $doc, $childre | |||
| if ( 'table' === $tag ) { | |||
| // Flush any accumulated inline runs. | |||
| if ( ! empty( $current_runs ) ) { | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr ); | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr, $alignment ); | |||
| $current_runs = array(); | |||
| } | |||
| // Convert nested table. | |||
| @@ -2053,12 +2171,17 @@ private static function convert_cell_content_to_docx( DOMDocument $doc, $childre | |||
| if ( 'p' === $tag || 'div' === $tag ) { | |||
| // Flush any accumulated inline runs. | |||
| if ( ! empty( $current_runs ) ) { | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr ); | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr, $alignment ); | |||
| $current_runs = array(); | |||
| } | |||
| // Extract paragraph-specific alignment or use cell alignment. | |||
| $p_alignment = self::extract_text_alignment( $child ); | |||
| if ( null === $p_alignment ) { | |||
| $p_alignment = $alignment; | |||
| } | |||
| // Convert paragraph content. | |||
| $p_runs = self::collect_runs_from_children( $doc, $child->childNodes, $base_rpr, $formatting, $relationships ); | |||
| $result[] = self::create_paragraph_from_runs( $doc, $p_runs, $base_rpr ); | |||
| $result[] = self::create_paragraph_from_runs( $doc, $p_runs, $base_rpr, $p_alignment ); | |||
| continue; | |||
| } | |||
|
|
|||
| @@ -2068,7 +2191,7 @@ private static function convert_cell_content_to_docx( DOMDocument $doc, $childre | |||
|
|
|||
| // Flush remaining inline runs. | |||
Check warning
Code scanning / PHPMD
Code Size Rules: CyclomaticComplexity Warning
Comment on lines
2071
to
2192
| @@ -2001,9 +2118,10 @@ private static function convert_table_node_to_docx( DOMDocument $doc, DOMElement | |||
| * @param DOMElement|null $base_rpr Base run properties. | |||
| * @param array<string,bool> $formatting Formatting flags. | |||
| * @param array<string,mixed>|null $relationships Relationships context. | |||
| * @param string|null $alignment Cell alignment (left, center, right, justify). | |||
| * @return array<int,DOMElement> Array of paragraph elements. | |||
| */ | |||
| private static function convert_cell_content_to_docx( DOMDocument $doc, $children, $base_rpr, array $formatting, &$relationships ) { | |||
| private static function convert_cell_content_to_docx( DOMDocument $doc, $children, $base_rpr, array $formatting, &$relationships, $alignment = null ) { | |||
| $result = array(); | |||
| $current_runs = array(); | |||
|
|
|||
| @@ -2027,7 +2145,7 @@ private static function convert_cell_content_to_docx( DOMDocument $doc, $childre | |||
| if ( 'ul' === $tag || 'ol' === $tag ) { | |||
| // Flush any accumulated inline runs. | |||
| if ( ! empty( $current_runs ) ) { | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr ); | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr, $alignment ); | |||
| $current_runs = array(); | |||
| } | |||
| // Convert list to paragraphs. | |||
| @@ -2039,7 +2157,7 @@ private static function convert_cell_content_to_docx( DOMDocument $doc, $childre | |||
| if ( 'table' === $tag ) { | |||
| // Flush any accumulated inline runs. | |||
| if ( ! empty( $current_runs ) ) { | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr ); | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr, $alignment ); | |||
| $current_runs = array(); | |||
| } | |||
| // Convert nested table. | |||
| @@ -2053,12 +2171,17 @@ private static function convert_cell_content_to_docx( DOMDocument $doc, $childre | |||
| if ( 'p' === $tag || 'div' === $tag ) { | |||
| // Flush any accumulated inline runs. | |||
| if ( ! empty( $current_runs ) ) { | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr ); | |||
| $result[] = self::create_paragraph_from_runs( $doc, $current_runs, $base_rpr, $alignment ); | |||
| $current_runs = array(); | |||
| } | |||
| // Extract paragraph-specific alignment or use cell alignment. | |||
| $p_alignment = self::extract_text_alignment( $child ); | |||
| if ( null === $p_alignment ) { | |||
| $p_alignment = $alignment; | |||
| } | |||
| // Convert paragraph content. | |||
| $p_runs = self::collect_runs_from_children( $doc, $child->childNodes, $base_rpr, $formatting, $relationships ); | |||
| $result[] = self::create_paragraph_from_runs( $doc, $p_runs, $base_rpr ); | |||
| $result[] = self::create_paragraph_from_runs( $doc, $p_runs, $base_rpr, $p_alignment ); | |||
| continue; | |||
| } | |||
|
|
|||
| @@ -2068,7 +2191,7 @@ private static function convert_cell_content_to_docx( DOMDocument $doc, $childre | |||
|
|
|||
| // Flush remaining inline runs. | |||
Check warning
Code scanning / PHPMD
Code Size Rules: NPathComplexity Warning
Comment on lines
+603
to
+714
| public static function apply_odt_metadata( $odt_path, $metadata ) { | ||
| if ( empty( $metadata ) || ! is_array( $metadata ) ) { | ||
| return true; | ||
| } | ||
|
|
||
| // Check if any metadata value is non-empty. | ||
| $has_values = false; | ||
| foreach ( $metadata as $value ) { | ||
| if ( ! empty( $value ) ) { | ||
| $has_values = true; | ||
| break; | ||
| } | ||
| } | ||
| if ( ! $has_values ) { | ||
| return true; | ||
| } | ||
|
|
||
| if ( ! class_exists( 'ZipArchive' ) ) { | ||
| return new WP_Error( 'documentate_odt_zip_missing', __( 'ZipArchive is not available for metadata.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $zip = new ZipArchive(); | ||
| if ( true !== $zip->open( $odt_path ) ) { | ||
| return new WP_Error( 'documentate_odt_open_failed', __( 'Could not open the ODT file for metadata.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $xml = $zip->getFromName( 'meta.xml' ); | ||
| if ( false === $xml ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_meta_missing', __( 'meta.xml not found in ODT.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $dom = self::create_xml_document( $xml ); | ||
| if ( ! $dom ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_meta_parse', __( 'Could not parse meta.xml.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $xpath = new DOMXPath( $dom ); | ||
| $xpath->registerNamespace( 'office', self::ODF_OFFICE_NS ); | ||
| $xpath->registerNamespace( 'dc', self::DC_NS ); | ||
| $xpath->registerNamespace( 'meta', self::ODF_META_NS ); | ||
|
|
||
| // Find office:meta element. | ||
| $office_meta_list = $xpath->query( '//office:meta' ); | ||
| if ( 0 === $office_meta_list->length ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_meta_element', __( 'office:meta element not found.', 'documentate' ) ); | ||
| } | ||
| $office_meta = $office_meta_list->item( 0 ); | ||
|
|
||
| // Helper to set or update an element. | ||
| $set_element = function ( $ns_uri, $local_name, $value ) use ( $dom, $xpath, $office_meta ) { | ||
| if ( empty( $value ) ) { | ||
| return; | ||
| } | ||
| $prefix = 'dc' === substr( $local_name, 0, 2 ) || in_array( $local_name, array( 'title', 'subject', 'creator' ), true ) ? 'dc' : 'meta'; | ||
| $query = ".//{$prefix}:{$local_name}"; | ||
| $nodes = $xpath->query( $query, $office_meta ); | ||
|
|
||
| if ( $nodes->length > 0 ) { | ||
| // Update existing element. | ||
| $nodes->item( 0 )->textContent = $value; | ||
| } else { | ||
| // Create new element. | ||
| $new_el = $dom->createElementNS( $ns_uri, "{$prefix}:{$local_name}" ); | ||
| $new_el->appendChild( $dom->createTextNode( $value ) ); | ||
| $office_meta->appendChild( $new_el ); | ||
| } | ||
| }; | ||
|
|
||
| // Set title. | ||
| if ( ! empty( $metadata['title'] ) ) { | ||
| $set_element( self::DC_NS, 'title', $metadata['title'] ); | ||
| } | ||
|
|
||
| // Set subject. | ||
| if ( ! empty( $metadata['subject'] ) ) { | ||
| $set_element( self::DC_NS, 'subject', $metadata['subject'] ); | ||
| } | ||
|
|
||
| // Set creator (author) - both initial-creator (LibreOffice author) and dc:creator. | ||
| if ( ! empty( $metadata['author'] ) ) { | ||
| $set_element( self::ODF_META_NS, 'initial-creator', $metadata['author'] ); | ||
| $set_element( self::DC_NS, 'creator', $metadata['author'] ); | ||
| } | ||
|
|
||
| // Set keywords - remove existing and add new ones. | ||
| if ( ! empty( $metadata['keywords'] ) ) { | ||
| // Remove existing keyword elements. | ||
| $existing_keywords = $xpath->query( './/meta:keyword', $office_meta ); | ||
| foreach ( $existing_keywords as $kw ) { | ||
| $office_meta->removeChild( $kw ); | ||
| } | ||
|
|
||
| // Add new keyword elements. | ||
| $keywords_array = array_map( 'trim', explode( ',', $metadata['keywords'] ) ); | ||
| $keywords_array = array_filter( $keywords_array ); | ||
| foreach ( $keywords_array as $keyword ) { | ||
| $kw_el = $dom->createElementNS( self::ODF_META_NS, 'meta:keyword' ); | ||
| $kw_el->appendChild( $dom->createTextNode( $keyword ) ); | ||
| $office_meta->appendChild( $kw_el ); | ||
| } | ||
| } | ||
|
|
||
| // Save updated meta.xml. | ||
| $updated_xml = $dom->saveXML(); | ||
| $zip->addFromString( 'meta.xml', $updated_xml ); | ||
| $zip->close(); | ||
|
|
||
| return true; | ||
| } |
Check warning
Code scanning / PHPMD
Code Size Rules: CyclomaticComplexity Warning
Comment on lines
+603
to
+714
| public static function apply_odt_metadata( $odt_path, $metadata ) { | ||
| if ( empty( $metadata ) || ! is_array( $metadata ) ) { | ||
| return true; | ||
| } | ||
|
|
||
| // Check if any metadata value is non-empty. | ||
| $has_values = false; | ||
| foreach ( $metadata as $value ) { | ||
| if ( ! empty( $value ) ) { | ||
| $has_values = true; | ||
| break; | ||
| } | ||
| } | ||
| if ( ! $has_values ) { | ||
| return true; | ||
| } | ||
|
|
||
| if ( ! class_exists( 'ZipArchive' ) ) { | ||
| return new WP_Error( 'documentate_odt_zip_missing', __( 'ZipArchive is not available for metadata.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $zip = new ZipArchive(); | ||
| if ( true !== $zip->open( $odt_path ) ) { | ||
| return new WP_Error( 'documentate_odt_open_failed', __( 'Could not open the ODT file for metadata.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $xml = $zip->getFromName( 'meta.xml' ); | ||
| if ( false === $xml ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_meta_missing', __( 'meta.xml not found in ODT.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $dom = self::create_xml_document( $xml ); | ||
| if ( ! $dom ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_meta_parse', __( 'Could not parse meta.xml.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $xpath = new DOMXPath( $dom ); | ||
| $xpath->registerNamespace( 'office', self::ODF_OFFICE_NS ); | ||
| $xpath->registerNamespace( 'dc', self::DC_NS ); | ||
| $xpath->registerNamespace( 'meta', self::ODF_META_NS ); | ||
|
|
||
| // Find office:meta element. | ||
| $office_meta_list = $xpath->query( '//office:meta' ); | ||
| if ( 0 === $office_meta_list->length ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_meta_element', __( 'office:meta element not found.', 'documentate' ) ); | ||
| } | ||
| $office_meta = $office_meta_list->item( 0 ); | ||
|
|
||
| // Helper to set or update an element. | ||
| $set_element = function ( $ns_uri, $local_name, $value ) use ( $dom, $xpath, $office_meta ) { | ||
| if ( empty( $value ) ) { | ||
| return; | ||
| } | ||
| $prefix = 'dc' === substr( $local_name, 0, 2 ) || in_array( $local_name, array( 'title', 'subject', 'creator' ), true ) ? 'dc' : 'meta'; | ||
| $query = ".//{$prefix}:{$local_name}"; | ||
| $nodes = $xpath->query( $query, $office_meta ); | ||
|
|
||
| if ( $nodes->length > 0 ) { | ||
| // Update existing element. | ||
| $nodes->item( 0 )->textContent = $value; | ||
| } else { | ||
| // Create new element. | ||
| $new_el = $dom->createElementNS( $ns_uri, "{$prefix}:{$local_name}" ); | ||
| $new_el->appendChild( $dom->createTextNode( $value ) ); | ||
| $office_meta->appendChild( $new_el ); | ||
| } | ||
| }; | ||
|
|
||
| // Set title. | ||
| if ( ! empty( $metadata['title'] ) ) { | ||
| $set_element( self::DC_NS, 'title', $metadata['title'] ); | ||
| } | ||
|
|
||
| // Set subject. | ||
| if ( ! empty( $metadata['subject'] ) ) { | ||
| $set_element( self::DC_NS, 'subject', $metadata['subject'] ); | ||
| } | ||
|
|
||
| // Set creator (author) - both initial-creator (LibreOffice author) and dc:creator. | ||
| if ( ! empty( $metadata['author'] ) ) { | ||
| $set_element( self::ODF_META_NS, 'initial-creator', $metadata['author'] ); | ||
| $set_element( self::DC_NS, 'creator', $metadata['author'] ); | ||
| } | ||
|
|
||
| // Set keywords - remove existing and add new ones. | ||
| if ( ! empty( $metadata['keywords'] ) ) { | ||
| // Remove existing keyword elements. | ||
| $existing_keywords = $xpath->query( './/meta:keyword', $office_meta ); | ||
| foreach ( $existing_keywords as $kw ) { | ||
| $office_meta->removeChild( $kw ); | ||
| } | ||
|
|
||
| // Add new keyword elements. | ||
| $keywords_array = array_map( 'trim', explode( ',', $metadata['keywords'] ) ); | ||
| $keywords_array = array_filter( $keywords_array ); | ||
| foreach ( $keywords_array as $keyword ) { | ||
| $kw_el = $dom->createElementNS( self::ODF_META_NS, 'meta:keyword' ); | ||
| $kw_el->appendChild( $dom->createTextNode( $keyword ) ); | ||
| $office_meta->appendChild( $kw_el ); | ||
| } | ||
| } | ||
|
|
||
| // Save updated meta.xml. | ||
| $updated_xml = $dom->saveXML(); | ||
| $zip->addFromString( 'meta.xml', $updated_xml ); | ||
| $zip->close(); | ||
|
|
||
| return true; | ||
| } |
Check warning
Code scanning / PHPMD
Code Size Rules: NPathComplexity Warning
Comment on lines
+726
to
+821
| public static function apply_docx_metadata( $docx_path, $metadata ) { | ||
| if ( empty( $metadata ) || ! is_array( $metadata ) ) { | ||
| return true; | ||
| } | ||
|
|
||
| // Check if any metadata value is non-empty. | ||
| $has_values = false; | ||
| foreach ( $metadata as $value ) { | ||
| if ( ! empty( $value ) ) { | ||
| $has_values = true; | ||
| break; | ||
| } | ||
| } | ||
| if ( ! $has_values ) { | ||
| return true; | ||
| } | ||
|
|
||
| if ( ! class_exists( 'ZipArchive' ) ) { | ||
| return new WP_Error( 'documentate_docx_zip_missing', __( 'ZipArchive is not available for metadata.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $zip = new ZipArchive(); | ||
| if ( true !== $zip->open( $docx_path ) ) { | ||
| return new WP_Error( 'documentate_docx_open_failed', __( 'Could not open the DOCX file for metadata.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $xml = $zip->getFromName( 'docProps/core.xml' ); | ||
| if ( false === $xml ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_core_missing', __( 'docProps/core.xml not found in DOCX.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $dom = self::create_xml_document( $xml ); | ||
| if ( ! $dom ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_core_parse', __( 'Could not parse core.xml.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $xpath = new DOMXPath( $dom ); | ||
| $xpath->registerNamespace( 'cp', self::CP_NS ); | ||
| $xpath->registerNamespace( 'dc', self::DC_NS ); | ||
|
|
||
| // Find cp:coreProperties element. | ||
| $core_props_list = $xpath->query( '//cp:coreProperties' ); | ||
| if ( 0 === $core_props_list->length ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_core_element', __( 'cp:coreProperties element not found.', 'documentate' ) ); | ||
| } | ||
| $core_props = $core_props_list->item( 0 ); | ||
|
|
||
| // Helper to set or update an element. | ||
| $set_element = function ( $ns_uri, $prefix, $local_name, $value ) use ( $dom, $xpath, $core_props ) { | ||
| if ( empty( $value ) ) { | ||
| return; | ||
| } | ||
| $query = ".//{$prefix}:{$local_name}"; | ||
| $nodes = $xpath->query( $query, $core_props ); | ||
|
|
||
| if ( $nodes->length > 0 ) { | ||
| // Update existing element. | ||
| $nodes->item( 0 )->textContent = $value; | ||
| } else { | ||
| // Create new element. | ||
| $new_el = $dom->createElementNS( $ns_uri, "{$prefix}:{$local_name}" ); | ||
| $new_el->appendChild( $dom->createTextNode( $value ) ); | ||
| $core_props->appendChild( $new_el ); | ||
| } | ||
| }; | ||
|
|
||
| // Set title. | ||
| if ( ! empty( $metadata['title'] ) ) { | ||
| $set_element( self::DC_NS, 'dc', 'title', $metadata['title'] ); | ||
| } | ||
|
|
||
| // Set subject. | ||
| if ( ! empty( $metadata['subject'] ) ) { | ||
| $set_element( self::DC_NS, 'dc', 'subject', $metadata['subject'] ); | ||
| } | ||
|
|
||
| // Set creator (author). | ||
| if ( ! empty( $metadata['author'] ) ) { | ||
| $set_element( self::DC_NS, 'dc', 'creator', $metadata['author'] ); | ||
| } | ||
|
|
||
| // Set keywords (DOCX uses cp:keywords as comma-separated string). | ||
| if ( ! empty( $metadata['keywords'] ) ) { | ||
| $set_element( self::CP_NS, 'cp', 'keywords', $metadata['keywords'] ); | ||
| } | ||
|
|
||
| // Save updated core.xml. | ||
| $updated_xml = $dom->saveXML(); | ||
| $zip->addFromString( 'docProps/core.xml', $updated_xml ); | ||
| $zip->close(); | ||
|
|
||
| return true; | ||
| } |
Check warning
Code scanning / PHPMD
Code Size Rules: CyclomaticComplexity Warning
Comment on lines
+726
to
+821
| public static function apply_docx_metadata( $docx_path, $metadata ) { | ||
| if ( empty( $metadata ) || ! is_array( $metadata ) ) { | ||
| return true; | ||
| } | ||
|
|
||
| // Check if any metadata value is non-empty. | ||
| $has_values = false; | ||
| foreach ( $metadata as $value ) { | ||
| if ( ! empty( $value ) ) { | ||
| $has_values = true; | ||
| break; | ||
| } | ||
| } | ||
| if ( ! $has_values ) { | ||
| return true; | ||
| } | ||
|
|
||
| if ( ! class_exists( 'ZipArchive' ) ) { | ||
| return new WP_Error( 'documentate_docx_zip_missing', __( 'ZipArchive is not available for metadata.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $zip = new ZipArchive(); | ||
| if ( true !== $zip->open( $docx_path ) ) { | ||
| return new WP_Error( 'documentate_docx_open_failed', __( 'Could not open the DOCX file for metadata.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $xml = $zip->getFromName( 'docProps/core.xml' ); | ||
| if ( false === $xml ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_core_missing', __( 'docProps/core.xml not found in DOCX.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $dom = self::create_xml_document( $xml ); | ||
| if ( ! $dom ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_core_parse', __( 'Could not parse core.xml.', 'documentate' ) ); | ||
| } | ||
|
|
||
| $xpath = new DOMXPath( $dom ); | ||
| $xpath->registerNamespace( 'cp', self::CP_NS ); | ||
| $xpath->registerNamespace( 'dc', self::DC_NS ); | ||
|
|
||
| // Find cp:coreProperties element. | ||
| $core_props_list = $xpath->query( '//cp:coreProperties' ); | ||
| if ( 0 === $core_props_list->length ) { | ||
| $zip->close(); | ||
| return new WP_Error( 'documentate_core_element', __( 'cp:coreProperties element not found.', 'documentate' ) ); | ||
| } | ||
| $core_props = $core_props_list->item( 0 ); | ||
|
|
||
| // Helper to set or update an element. | ||
| $set_element = function ( $ns_uri, $prefix, $local_name, $value ) use ( $dom, $xpath, $core_props ) { | ||
| if ( empty( $value ) ) { | ||
| return; | ||
| } | ||
| $query = ".//{$prefix}:{$local_name}"; | ||
| $nodes = $xpath->query( $query, $core_props ); | ||
|
|
||
| if ( $nodes->length > 0 ) { | ||
| // Update existing element. | ||
| $nodes->item( 0 )->textContent = $value; | ||
| } else { | ||
| // Create new element. | ||
| $new_el = $dom->createElementNS( $ns_uri, "{$prefix}:{$local_name}" ); | ||
| $new_el->appendChild( $dom->createTextNode( $value ) ); | ||
| $core_props->appendChild( $new_el ); | ||
| } | ||
| }; | ||
|
|
||
| // Set title. | ||
| if ( ! empty( $metadata['title'] ) ) { | ||
| $set_element( self::DC_NS, 'dc', 'title', $metadata['title'] ); | ||
| } | ||
|
|
||
| // Set subject. | ||
| if ( ! empty( $metadata['subject'] ) ) { | ||
| $set_element( self::DC_NS, 'dc', 'subject', $metadata['subject'] ); | ||
| } | ||
|
|
||
| // Set creator (author). | ||
| if ( ! empty( $metadata['author'] ) ) { | ||
| $set_element( self::DC_NS, 'dc', 'creator', $metadata['author'] ); | ||
| } | ||
|
|
||
| // Set keywords (DOCX uses cp:keywords as comma-separated string). | ||
| if ( ! empty( $metadata['keywords'] ) ) { | ||
| $set_element( self::CP_NS, 'cp', 'keywords', $metadata['keywords'] ); | ||
| } | ||
|
|
||
| // Save updated core.xml. | ||
| $updated_xml = $dom->saveXML(); | ||
| $zip->addFromString( 'docProps/core.xml', $updated_xml ); | ||
| $zip->close(); | ||
|
|
||
| return true; | ||
| } |
Check warning
Code scanning / PHPMD
Code Size Rules: NPathComplexity Warning
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the default document templates and demo data for the Documentate plugin, focusing on transitioning from generic test templates (
plantilla.odt/plantilla.docx) to domain-specific ones for "Resolución Administrativa" and related demo documents. It also ensures that tests and seed data use these new templates and document types.Template and Demo Document Updates
resolucion.odtanddemo-wp-documentate.docxthroughout the codebase and tests, making the demo and seed data more representative of real-world administrative resolutions. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]Demo Data and Document Type Seeding
Documentation
These changes make the plugin's seeded content and tests more realistic and domain-specific, improving both developer experience and demo usability.