joonda commented on code in PR #999:
URL: https://github.com/apache/poi/pull/999#discussion_r2701112836


##########
poi-ooxml/src/test/java/org/apache/poi/xslf/TestXSLFBugs.java:
##########
@@ -236,7 +236,7 @@ void bug61589() throws IOException {
             try (XMLSlideShow ppt3 = writeOutAndReadBack(dest)) {
                 XSLFSlide slide3 = ppt3.getSlides().get(0);
                 XSLFTextBox shape3 = (XSLFTextBox) slide3.getShapes().get(0);
-                XSLFTextParagraph p3 = shape3.getTextParagraphs().get(1);
+                XSLFTextParagraph p3 = shape3.getTextParagraphs().get(2);

Review Comment:
   Thank you for your thoughtful feedback.
   I completely agree that my previous approach of modifying the test indices 
was not the ideal way to solve this.
   
   I'll close this PR for now to explore a cleaner solution that ensures OOXML 
compliance without affecting the existing logic.
   
   I appreciate your patience and guidance.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to