]> source.dussan.org Git - poi.git/commitdiff
Remove more remnants of NPOIFSFileSystem, mostly in comments and variable names
authorDominik Stadler <centic@apache.org>
Tue, 19 Mar 2019 21:55:26 +0000 (21:55 +0000)
committerDominik Stadler <centic@apache.org>
Tue, 19 Mar 2019 21:55:26 +0000 (21:55 +0000)
This avoids confusion if people look at this code in the future without knowledge
about the historical transition from java.io to java.nio

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1855873 13f79535-47bb-0310-9956-ffa450edef68

32 files changed:
src/java/org/apache/poi/POIDocument.java
src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java
src/java/org/apache/poi/hssf/usermodel/HSSFWorkbookFactory.java
src/java/org/apache/poi/poifs/eventfilesystem/POIFSReader.java
src/java/org/apache/poi/poifs/filesystem/DirectoryNode.java
src/java/org/apache/poi/poifs/filesystem/DocumentFactoryHelper.java
src/java/org/apache/poi/poifs/macros/VBAMacroReader.java
src/java/org/apache/poi/sl/usermodel/SlideShowFactory.java
src/java/org/apache/poi/ss/usermodel/WorkbookFactory.java
src/ooxml/java/org/apache/poi/ooxml/extractor/ExtractorFactory.java
src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java
src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java
src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShow.java
src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShowFactory.java
src/scratchpad/testcases/org/apache/poi/hpbf/extractor/TestPublisherTextExtractor.java
src/scratchpad/testcases/org/apache/poi/hslf/extractor/TestExtractor.java
src/scratchpad/testcases/org/apache/poi/hslf/usermodel/TestBugs.java
src/scratchpad/testcases/org/apache/poi/hwpf/extractor/TestWordExtractor.java
src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestBugs.java
src/scratchpad/testcases/org/apache/poi/hwpf/usermodel/TestHWPFWrite.java
src/testcases/org/apache/poi/hpsf/basic/TestWrite.java
src/testcases/org/apache/poi/hpsf/extractor/TestHPSFPropertiesExtractor.java
src/testcases/org/apache/poi/hssf/extractor/TestOldExcelExtractor.java
src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java
src/testcases/org/apache/poi/hssf/usermodel/TestHSSFWorkbook.java
src/testcases/org/apache/poi/poifs/filesystem/TestDocument.java
src/testcases/org/apache/poi/poifs/filesystem/TestDocumentInputStream.java
src/testcases/org/apache/poi/poifs/filesystem/TestEntryUtils.java
src/testcases/org/apache/poi/poifs/filesystem/TestFileSystemBugs.java
src/testcases/org/apache/poi/poifs/filesystem/TestPOIFSStream.java
src/testcases/org/apache/poi/poifs/macros/TestVBAMacroReader.java
src/testcases/org/apache/poi/sl/usermodel/BaseTestSlideShowFactory.java

index 78015b4a954a03348cf50c5981b3d831dbd51d82..e7d8afa8d7ddb08f263da9d5496f196a866cf4f6 100644 (file)
@@ -236,7 +236,7 @@ public abstract class POIDocument implements Closeable {
     
     /**
      * Writes out the updated standard Document Information Properties (HPSF)
-     *  into the currently open NPOIFSFileSystem
+     *  into the currently open POIFSFileSystem
      * 
      * @throws IOException if an error when writing to the open
      *      {@link POIFSFileSystem} occurs
@@ -258,7 +258,7 @@ public abstract class POIDocument implements Closeable {
     }
     /**
      * Writes out the standard Document Information Properties (HPSF)
-     * @param outFS the NPOIFSFileSystem to write the properties into
+     * @param outFS the {@link POIFSFileSystem} to write the properties into
      * @param writtenEntries a list of POIFS entries to add the property names too
      * 
      * @throws IOException if an error when writing to the 
@@ -314,7 +314,7 @@ public abstract class POIDocument implements Closeable {
      *
      * @param name the (POIFS Level) name of the property to write
      * @param set the PropertySet to write out 
-     * @param outFS the NPOIFSFileSystem to write the property into
+     * @param outFS the {@link POIFSFileSystem} to write the property into
      * 
      * @throws IOException if an error when writing to the 
      *      {@link POIFSFileSystem} occurs
index 1f3485d8e49096d16fa852baaeeee19770e0aa6e..78a4e4fd9eee22cf8d250dc9af1885ce716778da 100644 (file)
@@ -393,7 +393,7 @@ public final class HSSFWorkbook extends POIDocument implements org.apache.poi.ss
      * @see #HSSFWorkbook(POIFSFileSystem)
      * @exception IOException if the stream cannot be read
      */
-    @SuppressWarnings("resource")   // NPOIFSFileSystem always closes the stream
+    @SuppressWarnings("resource")   // POIFSFileSystem always closes the stream
     public HSSFWorkbook(InputStream s, boolean preserveNodes)
             throws IOException
     {
index 96196707f6783d69b1becec91392856264f0b98b..af9b26fe425e28c10d64974f796b078f657e51a3 100644 (file)
@@ -41,7 +41,7 @@ public class HSSFWorkbookFactory extends WorkbookFactory {
     }
 
     /**
-     * Creates a HSSFWorkbook from the given NPOIFSFileSystem<p>
+     * Creates a HSSFWorkbook from the given {@link POIFSFileSystem}<p>
      * Note that in order to properly release resources the
      * Workbook should be closed after use.
      */
index 718709f51fa09dd65377f8d554ec418eb57f31e3..ee7c9ad4e08823ab01723b9dce2e890d1378c823 100644 (file)
@@ -78,9 +78,9 @@ public class POIFSReader
     }
 
     /**
-     * Read from a NPOIFSFileSystem and process the documents we get
+     * Read from a {@link POIFSFileSystem} and process the documents we get
      *
-     * @param poifs the NPOIFSFileSystem from which to read the data
+     * @param poifs the POIFSFileSystem from which to read the data
      *
      * @exception IOException on errors reading, or on invalid data
      */
index 47c7e7ff4b27ef1b6dd7518952f345b25a7de12c..e91da26fb5a88a314a455494d383e5f2575149b1 100644 (file)
@@ -49,8 +49,8 @@ public class DirectoryNode
     // Our list of entries, kept sorted to preserve order
     private final ArrayList<Entry> _entries = new ArrayList<>();
 
-    // the NPOIFSFileSytem we belong to
-    private final POIFSFileSystem _nfilesystem;
+    // the POIFSFileSystem we belong to
+    private final POIFSFileSystem _filesystem;
 
     // the path described by this document
     private final POIFSDocumentPath _path;
@@ -60,15 +60,15 @@ public class DirectoryNode
      * is intended strictly for the internal use of this package
      *
      * @param property the DirectoryProperty for this DirectoryEntry
-     * @param nfilesystem the NPOIFSFileSystem we belong to
+     * @param filesystem the {@link POIFSFileSystem} we belong to
      * @param parent the parent of this entry
      */
     DirectoryNode(final DirectoryProperty property,
-                  final POIFSFileSystem nfilesystem,
+                  final POIFSFileSystem filesystem,
                   final DirectoryNode parent)
     {
         super(property, parent);
-        this._nfilesystem = nfilesystem;
+        this._filesystem = filesystem;
 
         if (parent == null)
         {
@@ -91,7 +91,7 @@ public class DirectoryNode
             if (child.isDirectory())
             {
                 DirectoryProperty childDir = (DirectoryProperty) child;
-                childNode = new DirectoryNode(childDir, _nfilesystem, this);
+                childNode = new DirectoryNode(childDir, _filesystem, this);
             }
             else
             {
@@ -116,7 +116,7 @@ public class DirectoryNode
      */
     public POIFSFileSystem getFileSystem()
     {
-        return _nfilesystem;
+        return _filesystem;
     }
 
     /**
@@ -175,7 +175,7 @@ public class DirectoryNode
         DocumentNode     rval     = new DocumentNode(property, this);
 
         (( DirectoryProperty ) getProperty()).addChild(property);
-        _nfilesystem.addDocument(document);
+        _filesystem.addDocument(document);
 
         _entries.add(rval);
         _byname.put(property.getName(), rval);
@@ -228,7 +228,7 @@ public class DirectoryNode
             _byname.remove(entry.getName());
 
             try {
-                _nfilesystem.remove(entry);
+                _filesystem.remove(entry);
             } catch (IOException e) {
                 // TODO Work out how to report this, given we can't change the method signature...
                 throw new RuntimeException(e);
@@ -349,7 +349,7 @@ public class DirectoryNode
                                         final InputStream stream)
         throws IOException
     {
-        return createDocument(new POIFSDocument(name, _nfilesystem, stream));
+        return createDocument(new POIFSDocument(name, _filesystem, stream));
     }
 
     /**
@@ -368,7 +368,7 @@ public class DirectoryNode
                                         final POIFSWriterListener writer)
         throws IOException
     {
-        return createDocument(new POIFSDocument(name, size, _nfilesystem, writer));
+        return createDocument(new POIFSDocument(name, size, _filesystem, writer));
     }
 
     /**
@@ -386,8 +386,8 @@ public class DirectoryNode
     {
         DirectoryProperty property = new DirectoryProperty(name);
 
-        DirectoryNode rval = new DirectoryNode(property, _nfilesystem, this);
-       _nfilesystem.addDirectory(property);
+        DirectoryNode rval = new DirectoryNode(property, _filesystem, this);
+       _filesystem.addDirectory(property);
 
         (( DirectoryProperty ) getProperty()).addChild(property);
         _entries.add(rval);
index e53f58475d2d82962d6162b0164f7ce2c5500790..9ac5e94018cdafa709bb3423853a81a737804a5e 100644 (file)
@@ -38,7 +38,7 @@ public final class DocumentFactoryHelper {
     }
 
     /**
-     * Wrap the OLE2 data in the NPOIFSFileSystem into a decrypted stream by using
+     * Wrap the OLE2 data in the {@link POIFSFileSystem} into a decrypted stream by using
      * the given password.
      *
      * @param fs The OLE2 stream for the document
@@ -48,7 +48,7 @@ public final class DocumentFactoryHelper {
      */
     public static InputStream getDecryptedStream(final POIFSFileSystem fs, String password)
     throws IOException {
-        // wrap the stream in a FilterInputStream to close the NPOIFSFileSystem
+        // wrap the stream in a FilterInputStream to close the POIFSFileSystem
         // as well when the resulting OPCPackage is closed
         return new FilterInputStream(getDecryptedStream(fs.getRoot(), password)) {
             @Override
index 8254b86d8a93e43525a38a5625084bd66e340298..cbb4ccba545133bd255422067ff4983c1545b534 100644 (file)
@@ -104,7 +104,7 @@ public class VBAMacroReader implements Closeable {
             while ((zipEntry = zis.getNextEntry()) != null) {
                 if (endsWithIgnoreCase(zipEntry.getName(), VBA_PROJECT_OOXML)) {
                     try {
-                        // Make a NPOIFS from the contents, and close the stream
+                        // Make a POIFSFileSystem from the contents, and close the stream
                         this.fs = new POIFSFileSystem(zis);
                         return;
                     } catch (IOException e) {
index 460e93bb23c275e108c7c3239b9dcd6e05bbe928..bda928cfceb6636ec2c4c1cf098160baefc515c5 100644 (file)
@@ -36,7 +36,7 @@ import org.apache.poi.util.IOUtils;
 
 public class SlideShowFactory {
     /**
-     * Creates a SlideShow from the given NPOIFSFileSystem.
+     * Creates a SlideShow from the given POIFSFileSystem.
      *
      * @param fs The {@link POIFSFileSystem} to read the document from
      *
@@ -52,7 +52,7 @@ public class SlideShowFactory {
     }
 
     /**
-     * Creates a SlideShow from the given NPOIFSFileSystem, which may
+     * Creates a SlideShow from the given POIFSFileSystem, which may
      * be password protected
      *
      * @param fs The {@link POIFSFileSystem} to read the document from
@@ -70,7 +70,7 @@ public class SlideShowFactory {
     }
 
     /**
-     * Creates a SlideShow from the given NPOIFSFileSystem.
+     * Creates a SlideShow from the given DirectoryNode.
      *
      * @param root The {@link DirectoryNode} to start reading the document from
      *
@@ -87,7 +87,7 @@ public class SlideShowFactory {
 
 
     /**
-     * Creates a SlideShow from the given NPOIFSFileSystem, which may
+     * Creates a SlideShow from the given DirectoryNode, which may
      * be password protected
      *
      * @param root The {@link DirectoryNode} to start reading the document from
index 45fa47d3ed9eac1764d4889bba96420961b69e3d..ca291a68a7b91413cefb52ea80ddfd17e659c492 100644 (file)
@@ -62,7 +62,7 @@ public class WorkbookFactory {
     }
 
     /**
-     * Creates a HSSFWorkbook from the given NPOIFSFileSystem<p>
+     * Creates a HSSFWorkbook from the given POIFSFileSystem<p>
      *
      * Note that in order to properly release resources the
      * Workbook should be closed after use.
@@ -78,7 +78,7 @@ public class WorkbookFactory {
     }
 
     /**
-     * Creates a Workbook from the given NPOIFSFileSystem, which may
+     * Creates a Workbook from the given POIFSFileSystem, which may
      *  be password protected
      *
      *  @param fs The {@link POIFSFileSystem} to read the document from
@@ -94,7 +94,7 @@ public class WorkbookFactory {
 
 
     /**
-     * Creates a Workbook from the given NPOIFSFileSystem.
+     * Creates a Workbook from the given DirectoryNode.
      *
      * @param root The {@link DirectoryNode} to start reading the document from
      *
@@ -108,7 +108,7 @@ public class WorkbookFactory {
 
 
     /**
-     * Creates a Workbook from the given NPOIFSFileSystem, which may
+     * Creates a Workbook from the given DirectoryNode, which may
      * be password protected
      *
      * @param root The {@link DirectoryNode} to start reading the document from
index 412cacfeacf44da7e5ef3409cc9e77e775678556..70360e64e50d305cb3c939f31398b6a97b15a674 100644 (file)
@@ -404,7 +404,7 @@ public final class ExtractorFactory {
         } finally {
             IOUtils.closeQuietly(is);
 
-            // also close the NPOIFSFileSystem here as we read all the data
+            // also close the POIFSFileSystem here as we read all the data
             // while decrypting
             fs.close();
         }
index bdc9b6ddc8b3004eef41e6509993013eeb459568..683ad8a0e61386dfddde14e6bc28679078be4875 100644 (file)
@@ -1505,7 +1505,7 @@ public final class TestXSSFBugs extends BaseTestBugzillaIssues {
 
     @Test
     public void bug55692_stream() throws IOException {
-        // Directly on a Stream, will go via NPOIFS and spot it's
+        // Directly on a Stream, will go via POIFS and spot it's
         //  actually a .xlsx file encrypted with the default password, and open
         Workbook wb = WorkbookFactory.create(
                 POIDataSamples.getPOIFSInstance().openResourceAsStream("protect.xlsx"));
@@ -1515,8 +1515,8 @@ public final class TestXSSFBugs extends BaseTestBugzillaIssues {
     }
 
     @Test
-    public void bug55692_npoifs() throws IOException {
-        // Via a NPOIFSFileSystem, will spot it's actually a .xlsx file
+    public void bug55692_poifs2() throws IOException {
+        // Via a POIFSFileSystem, will spot it's actually a .xlsx file
         //  encrypted with the default password, and open
         POIFSFileSystem fsNP = new POIFSFileSystem(
                 POIDataSamples.getPOIFSInstance().openResourceAsStream("protect.xlsx"));
index dc2ecc776a4e307ee324472f3ea7cb0fda26bab8..6ae0663eb2860fbaedf45761d84ecf8ba69355a3 100644 (file)
@@ -117,7 +117,7 @@ public final class PowerPointExtractor extends POIOLE2TextExtractor {
 
    /**
     * Creates a PowerPointExtractor, from a specific place
-    *  inside an open NPOIFSFileSystem
+    *  inside an open {@link POIFSFileSystem}
     *
     * @param dir the POIFS Directory containing the PowerPoint document
     */
index 2343ccc48911f6436336f92841db956ebaac5bc7..5cc28e4f543f6d951996fdc4bbc02a4692a0a712 100644 (file)
@@ -148,8 +148,8 @@ public final class HSLFSlideShow implements SlideShow<HSLFShape,HSLFTextParagrap
      * Constructs a Powerpoint document from an POIFSFileSystem.
      */
     @SuppressWarnings("resource")
-    public HSLFSlideShow(POIFSFileSystem npoifs) throws IOException {
-        this(new HSLFSlideShowImpl(npoifs));
+    public HSLFSlideShow(POIFSFileSystem poifs) throws IOException {
+        this(new HSLFSlideShowImpl(poifs));
     }
 
     /**
index c52124138ee5b168494d5e19805d5f603d4a9c12..c25b13f2bd46414f3ce4ec8fc3e7e7ae325cc1ec 100644 (file)
@@ -32,7 +32,7 @@ import org.apache.poi.util.Internal;
 @Internal
 public class HSLFSlideShowFactory extends SlideShowFactory {
     /**
-     * Creates a HSLFSlideShow from the given NPOIFSFileSystem<p>
+     * Creates a HSLFSlideShow from the given {@link POIFSFileSystem}<p>
      * Note that in order to properly release resources the
      * SlideShow should be closed after use.
      */
index 06b7c98a817f2a6c514091318111ff8c832afa5b..c6353dae6d7922279945c5a0a8624becbfde31fb 100644 (file)
@@ -98,11 +98,11 @@ public final class TestPublisherTextExtractor {
         // And with NPOIFS
         sample = _samples.openResourceAsStream("Sample.pub");
         POIFSFileSystem fs = new POIFSFileSystem(sample);
-        HPBFDocument docNPOIFS = new HPBFDocument(fs);
-        ext = new PublisherTextExtractor(docNPOIFS);
+        HPBFDocument docPOIFS = new HPBFDocument(fs);
+        ext = new PublisherTextExtractor(docPOIFS);
         assertEquals(SAMPLE_TEXT, ext.getText());
         ext.close();
-        docNPOIFS.close();
+        docPOIFS.close();
         fs.close();
         sample.close();
 
index 9fa47b410719e5465b0f1ddd4643955ef6d5edbf..9cb01825e6dc65ddb485512f59b0b526b2a4a5dd 100644 (file)
@@ -394,9 +394,9 @@ public final class TestExtractor {
     public void testDifferentPOIFS() throws IOException {
         // Open the two filesystems
         File pptFile = slTests.getFile("basic_test_ppt_file.ppt");
-        try (final POIFSFileSystem npoifs = new POIFSFileSystem(pptFile, true)) {
+        try (final POIFSFileSystem poifs = new POIFSFileSystem(pptFile, true)) {
             // Open directly
-            try (SlideShow<?,?> ppt = SlideShowFactory.create(npoifs.getRoot());
+            try (SlideShow<?,?> ppt = SlideShowFactory.create(poifs.getRoot());
                 SlideShowExtractor<?,?> extractor = new SlideShowExtractor<>(ppt)) {
                 assertEquals(EXPECTED_PAGE1+EXPECTED_PAGE2, extractor.getText());
             }
index f2a111ea3bbfd74527dcbc38c4cd4657c41916f1..73756e5332833e6969d0cf9f39e3f4469a8d4c2f 100644 (file)
@@ -992,15 +992,15 @@ public final class TestBugs {
     //It isn't pretty, but it works...
     private Map<String, String> getMacrosFromHSLF(String fileName) throws IOException {
         InputStream is = null;
-        POIFSFileSystem npoifs = null;
+        POIFSFileSystem poifs = null;
         try {
             is = new FileInputStream(POIDataSamples.getSlideShowInstance().getFile(fileName));
-            npoifs = new POIFSFileSystem(is);
-            //TODO: should we run the VBAMacroReader on this npoifs?
+            poifs = new POIFSFileSystem(is);
+            //TODO: should we run the VBAMacroReader on this poifs?
             //TBD: We know that ppt typically don't store macros in the regular place,
             //but _can_ they?
 
-            HSLFSlideShow ppt = new HSLFSlideShow(npoifs);
+            HSLFSlideShow ppt = new HSLFSlideShow(poifs);
 
             //get macro persist id
             DocInfoListContainer list = (DocInfoListContainer)ppt.getDocumentRecord().findFirstOfType(RecordTypes.List.typeID);
@@ -1018,7 +1018,7 @@ public final class TestBugs {
             ppt.close();
 
         } finally {
-            IOUtils.closeQuietly(npoifs);
+            IOUtils.closeQuietly(poifs);
             IOUtils.closeQuietly(is);
         }
         return null;
index ea6fc3966e9dcc7219499426b1727380a0e1cebc..5d919c7a1500a57c0d3a5007e9e95818311a69ca 100644 (file)
@@ -343,9 +343,9 @@ public final class TestWordExtractor {
     public void testDifferentPOIFS() throws Exception {
        // Open the two filesystems
        File file = docTests.getFile("test2.doc");
-       try (POIFSFileSystem npoifs = new POIFSFileSystem(file, true)) {
+       try (POIFSFileSystem poifs = new POIFSFileSystem(file, true)) {
 
-           DirectoryNode dir = npoifs.getRoot();
+           DirectoryNode dir = poifs.getRoot();
 
            // Open directly
            @SuppressWarnings("resource")
index bb42d771258712e70ea77cdbae62c07f4c8ff536..5495fa7d6a6a2aefa6c7ab83e9def859d91cb99e 100644 (file)
@@ -606,7 +606,7 @@ public class TestBugs{
     }
 
     /**
-     * [RESOLVED FIXED] Bug 51671 - HWPFDocument.write based on NPOIFSFileSystem
+     * [RESOLVED FIXED] Bug 51671 - HWPFDocument.write based on POIFSFileSystem
      * throws a NullPointerException
      */
     @Test
@@ -614,9 +614,9 @@ public class TestBugs{
     {
         InputStream is = POIDataSamples.getDocumentInstance()
                 .openResourceAsStream("empty.doc");
-        try (POIFSFileSystem npoifsFileSystem = new POIFSFileSystem(is)) {
+        try (POIFSFileSystem poifsFileSystem = new POIFSFileSystem(is)) {
             HWPFDocument hwpfDocument = new HWPFDocument(
-                    npoifsFileSystem.getRoot());
+                    poifsFileSystem.getRoot());
             hwpfDocument.write(new ByteArrayOutputStream());
             hwpfDocument.close();
         }
index 8c57ffdd34b9278e79fdaca4af87ed2405436df0..19504bd1087d324c2e5b7d4c129a9449db9eced0 100644 (file)
@@ -135,7 +135,7 @@ public final class TestHWPFWrite extends HWPFTestCase {
     }
     
     @Test(expected=IllegalStateException.class)
-    public void testInvalidInPlaceWriteNPOIFS() throws Exception {
+    public void testInvalidInPlaceWritePOIFS() throws Exception {
         // Can't work for Read-Only files
         POIFSFileSystem fs = new POIFSFileSystem(SAMPLES.getFile("SampleDoc.doc"), true);
         HWPFDocument doc = new HWPFDocument(fs.getRoot());
index 235ced34329df72a116a9f99c22360df28b4f2c2..ff1fb0be4503f4b56d1adbfe593d64a2555b6080 100644 (file)
@@ -495,11 +495,11 @@ public class TestWrite {
     }
 
     /**
-     * Tests that when using NPOIFS, we can do an in-place write
+     * Tests that when using POIFS, we can do an in-place write
      *  without needing to stream in + out the whole kitchen sink
      */
     @Test
-    public void inPlaceNPOIFSWrite() throws Exception {
+    public void inPlacePOIFSWrite() throws Exception {
         // We need to work on a File for in-place changes, so create a temp one
         final File copy = TempFile.createTempFile("Test-HPSF", "ole2");
         copy.deleteOnExit();
index 47ecf04d7ce5b1b60b440936b791b55a28e635c5..e510df94f838b087b3eece180c4155f9c140093a 100644 (file)
@@ -183,9 +183,9 @@ public final class TestHPSFPropertiesExtractor {
 
        @Test
        public void test61300Extractor() throws IOException {
-               try (POIFSFileSystem npoifs = new POIFSFileSystem(
+               try (POIFSFileSystem poifs = new POIFSFileSystem(
                                POIDataSamples.getPOIFSInstance().getFile("61300.bin"))) {
-                       HPSFPropertiesExtractor ext = new HPSFPropertiesExtractor(npoifs);
+                       HPSFPropertiesExtractor ext = new HPSFPropertiesExtractor(poifs);
                        assertContains(ext.getText(), "PID_CODEPAGE = 1252");
                }
        }
index 323b59fcb90c9df60b4d3632bfbe10fa1a0d7995..acfc68cfc8498c1dfa45909136f337b7144afc21 100644 (file)
@@ -283,7 +283,7 @@ public final class TestOldExcelExtractor {
     }
 
     @Test
-    public void testNPOIFSFileSystem() throws IOException {
+    public void testPOIFSFileSystem() throws IOException {
         File file = HSSFTestDataSamples.getSampleFile("FormulaRefs.xls");
         try (POIFSFileSystem fs = new POIFSFileSystem(file)) {
             OldExcelExtractor extractor = new OldExcelExtractor(fs);
index 7153e3f8597de352e0ac9eee81b2b2491cd4673f..9d9c277def30fc4f5c30a1b7610a2c1a5715eb0f 100644 (file)
@@ -2331,34 +2331,29 @@ public final class TestBugs extends BaseTestBugzillaIssues {
      * File with exactly 256 data blocks (+header block)
      * shouldn't break on POIFS loading
      */
-    @SuppressWarnings("resource")
     @Test
     public void bug51461() throws Exception {
         byte[] data = HSSFITestDataProvider.instance.getTestDataFileContent("51461.xls");
 
         HSSFWorkbook wbPOIFS = new HSSFWorkbook(new POIFSFileSystem(
                 new ByteArrayInputStream(data)).getRoot(), false);
-        HSSFWorkbook wbNPOIFS = new HSSFWorkbook(new POIFSFileSystem(
+        HSSFWorkbook wbPOIFS2 = new HSSFWorkbook(new POIFSFileSystem(
                 new ByteArrayInputStream(data)).getRoot(), false);
 
         assertEquals(2, wbPOIFS.getNumberOfSheets());
-        assertEquals(2, wbNPOIFS.getNumberOfSheets());
+        assertEquals(2, wbPOIFS2.getNumberOfSheets());
     }
 
-    /**
-     * Large row numbers and NPOIFS vs POIFS
-     */
-    @SuppressWarnings("resource")
     @Test
     public void bug51535() throws Exception {
         byte[] data = HSSFITestDataProvider.instance.getTestDataFileContent("51535.xls");
 
         HSSFWorkbook wbPOIFS = new HSSFWorkbook(new POIFSFileSystem(
                 new ByteArrayInputStream(data)).getRoot(), false);
-        HSSFWorkbook wbNPOIFS = new HSSFWorkbook(new POIFSFileSystem(
+        HSSFWorkbook wbPOIFS2 = new HSSFWorkbook(new POIFSFileSystem(
                 new ByteArrayInputStream(data)).getRoot(), false);
 
-        for (HSSFWorkbook wb : new HSSFWorkbook[]{wbPOIFS, wbNPOIFS}) {
+        for (HSSFWorkbook wb : new HSSFWorkbook[]{wbPOIFS, wbPOIFS2}) {
             assertEquals(3, wb.getNumberOfSheets());
 
             // Check directly
@@ -3095,10 +3090,10 @@ public final class TestBugs extends BaseTestBugzillaIssues {
 
     @Test(expected = RuntimeException.class)
     public void test61300() throws Exception {
-        POIFSFileSystem npoifs = new POIFSFileSystem(HSSFTestDataSamples.openSampleFileStream("61300.xls"));
+        POIFSFileSystem poifs = new POIFSFileSystem(HSSFTestDataSamples.openSampleFileStream("61300.xls"));
 
         DocumentEntry entry =
-                (DocumentEntry) npoifs.getRoot().getEntry(SummaryInformation.DEFAULT_STREAM_NAME);
+                (DocumentEntry) poifs.getRoot().getEntry(SummaryInformation.DEFAULT_STREAM_NAME);
 
         // this will throw an Exception "RuntimeException: Can't read negative number of bytes"
         new PropertySet(new DocumentInputStream(entry));
index 0167ccc3d031d3ff4a8e1ec72f1a34a3a88c8cae..e3f4aeaef24f5b1e54b979afe16e3ae13d5efcc8 100644 (file)
@@ -621,8 +621,8 @@ public final class TestHSSFWorkbook extends BaseTestWorkbook {
        DirectoryNode[] files = new DirectoryNode[2];
         try (POIFSFileSystem poifsFileSystem = new POIFSFileSystem(HSSFTestDataSamples.openSampleFileStream("Simple.xls"))) {
             files[0] = poifsFileSystem.getRoot();
-            try (POIFSFileSystem npoifsFileSystem = new POIFSFileSystem(HSSFTestDataSamples.getSampleFile("Simple.xls"))) {
-                files[1] = npoifsFileSystem.getRoot();
+            try (POIFSFileSystem poifsFileSystem2 = new POIFSFileSystem(HSSFTestDataSamples.getSampleFile("Simple.xls"))) {
+                files[1] = poifsFileSystem2.getRoot();
 
                 // Open without preserving nodes
                 for (DirectoryNode dir : files) {
@@ -653,8 +653,8 @@ public final class TestHSSFWorkbook extends BaseTestWorkbook {
        DirectoryNode[] files = new DirectoryNode[2];
         try (POIFSFileSystem poifsFileSystem = new POIFSFileSystem(HSSFTestDataSamples.openSampleFileStream("WithEmbeddedObjects.xls"))) {
             files[0] = poifsFileSystem.getRoot();
-            try (POIFSFileSystem npoifsFileSystem = new POIFSFileSystem(HSSFTestDataSamples.getSampleFile("WithEmbeddedObjects.xls"))) {
-                files[1] = npoifsFileSystem.getRoot();
+            try (POIFSFileSystem poifsFileSystem2 = new POIFSFileSystem(HSSFTestDataSamples.getSampleFile("WithEmbeddedObjects.xls"))) {
+                files[1] = poifsFileSystem2.getRoot();
 
                 // Check the embedded parts
                 for (DirectoryNode root : files) {
@@ -682,19 +682,18 @@ public final class TestHSSFWorkbook extends BaseTestWorkbook {
     }
 
     /**
-     * Checks that we can open a workbook with NPOIFS, and write it out
+     * Checks that we can open a workbook with POIFS, and write it out
      *  again (via POIFS) and have it be valid
      */
     @Test
-    public void writeWorkbookFromNPOIFS() throws IOException {
+    public void writeWorkbookFromPOIFS() throws IOException {
         try (InputStream is = HSSFTestDataSamples.openSampleFileStream("WithEmbeddedObjects.xls");
-             POIFSFileSystem fs = new POIFSFileSystem(is)) {
-            // Start as NPOIFS
+            POIFSFileSystem fs = new POIFSFileSystem(is)) {
+
             HSSFWorkbook wb = new HSSFWorkbook(fs.getRoot(), true);
             assertEquals(3, wb.getNumberOfSheets());
             assertEquals("Root xls", wb.getSheetAt(0).getRow(0).getCell(0).getStringCellValue());
 
-            // Will switch to POIFS
             HSSFWorkbook wbBack = HSSFTestDataSamples.writeOutAndReadBack(wb);
             assertEquals(3, wbBack.getNumberOfSheets());
             assertEquals("Root xls", wbBack.getSheetAt(0).getRow(0).getCell(0).getStringCellValue());
@@ -1151,11 +1150,7 @@ public final class TestHSSFWorkbook extends BaseTestWorkbook {
         final File file = POIDataSamples.getSpreadSheetInstance().getFile(filename);
         Workbook wb;
         
-        // File via POIFileStream (java.io)
-        wb = new HSSFWorkbook(new POIFSFileSystem(file));
-        assertCloseDoesNotModifyFile(filename, wb);
-        
-        // File via NPOIFileStream (java.nio)
+        // File via POIFileStream (java.nio)
         wb = new HSSFWorkbook(new POIFSFileSystem(file));
         assertCloseDoesNotModifyFile(filename, wb);
         
index 6ca6f614d9a961ab07368155bdccaf114b5bb125..59839672e32ba3ae28b25b1e6abf579f0757fa24 100644 (file)
@@ -43,7 +43,7 @@ public class TestDocument {
      * Integration test -- really about all we can do
      */
     @Test
-    public void testNPOIFSDocument() throws IOException {
+    public void testPOIFSDocument() throws IOException {
 
         try (POIFSFileSystem poifs = new POIFSFileSystem()) {
 
index 6f1470568536efbfebe6c1e8fa872dcd7094adb5..ba03fcaf69b958a9c3ed456064f4fe3f6f04d374 100644 (file)
@@ -55,20 +55,20 @@ public final class TestDocumentInputStream {
             _workbook_data[j] = (byte) (j * j);
         }
 
-        // Now create the NPOIFS Version
+        // Now create the POIFS Version
         byte[] _workbook_data_only = new byte[_workbook_size];
         System.arraycopy(_workbook_data, 0, _workbook_data_only, 0, _workbook_size);
 
-        POIFSFileSystem npoifs = new POIFSFileSystem();
+        POIFSFileSystem poifs = new POIFSFileSystem();
         // Make it easy when debugging to see what isn't the doc
         byte[] minus1 = new byte[512];
         Arrays.fill(minus1, (byte) -1);
-        npoifs.getBlockAt(-1).put(minus1);
-        npoifs.getBlockAt(0).put(minus1);
-        npoifs.getBlockAt(1).put(minus1);
+        poifs.getBlockAt(-1).put(minus1);
+        poifs.getBlockAt(0).put(minus1);
+        poifs.getBlockAt(1).put(minus1);
 
-        // Create the NPOIFS document
-        _workbook_n = (DocumentNode) npoifs.createDocument(
+        // Create the POIFS document
+        _workbook_n = (DocumentNode) poifs.createDocument(
                 new ByteArrayInputStream(_workbook_data_only),
                 "Workbook"
         );
@@ -440,12 +440,12 @@ public final class TestDocumentInputStream {
 
         DocumentInputStream stream;
 
-        try (POIFSFileSystem npoifs = new POIFSFileSystem(sample)) {
+        try (POIFSFileSystem poifs = new POIFSFileSystem(sample)) {
             // Ensure we have what we expect on the root
-            assertEquals(npoifs, npoifs.getRoot().getFileSystem());
+            assertEquals(poifs, poifs.getRoot().getFileSystem());
 
             // Check inside
-            DirectoryNode root = npoifs.getRoot();
+            DirectoryNode root = poifs.getRoot();
             // Top Level
             Entry top = root.getEntry("Contents");
             assertTrue(top.isDocumentEntry());
index adb6cabe4f48f8799e177c5d9eb2e4edbf435058..7dca078a3ea6ea433b95055bfe3455c53682dd84 100644 (file)
@@ -123,7 +123,7 @@ public class TestEntryUtils {
        assertTrue(EntryUtils.areDocumentsIdentical(entryA1, entryB1));
        
        
-       // Can work with NPOIFS + POIFS
+       // Can work with POIFS
        ByteArrayOutputStream tmpO = new ByteArrayOutputStream();
        fs.writeFilesystem(tmpO);
        
index dc220d7f4048104bf360c063c0341e9c42411b77..128a86aadc99246d8a925f304cb0c5bb72d83e94 100644 (file)
@@ -34,7 +34,7 @@ import org.junit.After;
 import org.junit.Test;
 
 /**
- * Tests bugs across both POIFSFileSystem and NPOIFSFileSystem
+ * Tests bugs for POIFSFileSystem
  */
 public final class TestFileSystemBugs {
     private static POIDataSamples _samples = POIDataSamples.getPOIFSInstance();
@@ -115,7 +115,7 @@ public final class TestFileSystemBugs {
      * Ensure that a file with a corrupted property in the
      *  properties table can still be loaded, and the remaining
      *  properties used
-     * Note - only works for NPOIFSFileSystem, POIFSFileSystem
+     * Note - only works for POIFSFileSystem
      *  can't cope with this level of corruption
      */
     @Test
index eb88186a8e692586e7ae7f28417bd349201b5c23..0f0eeba31f5670eec010bff9a2162081ca73763f 100644 (file)
@@ -2605,11 +2605,11 @@ public final class TestPOIFSStream {
    }
 
    /**
-    * Test that we can read a file with NPOIFS, create a new NPOIFS instance,
+    * Test that we can read a file with POIFS, create a new POIFS instance,
     *  write it out, read it with POIFS, and see the original data
     */
    @Test
-   public void NPOIFSReadCopyWritePOIFSRead() throws IOException {
+   public void POIFSReadCopyWritePOIFSRead() throws IOException {
       File testFile = POIDataSamples.getSpreadSheetInstance().getFile("Simple.xls");
       POIFSFileSystem src = new POIFSFileSystem(testFile);
        byte[] wbDataExp = IOUtils.toByteArray(src.createDocumentInputStream("Workbook"));
index c6853f156c1a702e7fc7afbf26afb8fb0b854cd2..caf12223597e716aa90fd7121d76ca7c7cc9dc46 100644 (file)
@@ -148,24 +148,24 @@ public class TestVBAMacroReader {
         fromFile(POIDataSamples.getDiagramInstance(), "SimpleMacro.vsdm");
     }
 
-    //////////////////////////////// From NPOIFS /////////////////////////////
+    //////////////////////////////// From POIFS /////////////////////////////
     @Test
-    public void HSSFFromNPOIFS() throws Exception {
-        fromNPOIFS(POIDataSamples.getSpreadSheetInstance(), "SimpleMacro.xls");
+    public void HSSFFromPOIFS() throws Exception {
+        fromPOIFS(POIDataSamples.getSpreadSheetInstance(), "SimpleMacro.xls");
     }
     @Ignore("bug 59302: Found 0 macros")
     @Test
-    public void HSLFFromNPOIFS() throws Exception {
-        fromNPOIFS(POIDataSamples.getSlideShowInstance(), "SimpleMacro.ppt");
+    public void HSLFFromPOIFS() throws Exception {
+        fromPOIFS(POIDataSamples.getSlideShowInstance(), "SimpleMacro.ppt");
     }
     @Test
-    public void HWPFFromNPOIFS() throws Exception {
-        fromNPOIFS(POIDataSamples.getDocumentInstance(), "SimpleMacro.doc");
+    public void HWPFFromPOIFS() throws Exception {
+        fromPOIFS(POIDataSamples.getDocumentInstance(), "SimpleMacro.doc");
     }
     @Ignore("Found 0 macros")
     @Test
-    public void HDGFFromNPOIFS() throws Exception {
-        fromNPOIFS(POIDataSamples.getDiagramInstance(), "SimpleMacro.vsd");
+    public void HDGFFromPOIFS() throws Exception {
+        fromPOIFS(POIDataSamples.getDiagramInstance(), "SimpleMacro.vsd");
     }
 
     private void fromFile(POIDataSamples dataSamples, String filename) throws IOException {
@@ -183,7 +183,7 @@ public class TestVBAMacroReader {
         }
     }
 
-    private void fromNPOIFS(POIDataSamples dataSamples, String filename) throws IOException {
+    private void fromPOIFS(POIDataSamples dataSamples, String filename) throws IOException {
         File f = dataSamples.getFile(filename);
         try (POIFSFileSystem fs = new POIFSFileSystem(f)) {
             try (VBAMacroReader r = new VBAMacroReader(fs)) {
index 702c0e845c3efd8eec4f50ce195cfc886c5092a1..d62d787d94d6aaf19913c2390c1c8a43e389d8e9 100644 (file)
@@ -55,12 +55,12 @@ public abstract class BaseTestSlideShowFactory {
     @SuppressWarnings("resource")
     protected static void testFactoryFromNative(String file) throws Exception {
         SlideShow<?,?> ss;
-        // from NPOIFS
+        // from POIFS
         if (file.endsWith(".ppt")) {
-            POIFSFileSystem npoifs = new POIFSFileSystem(fromFile(file));
-            ss = SlideShowFactory.create(npoifs);
+            POIFSFileSystem poifs = new POIFSFileSystem(fromFile(file));
+            ss = SlideShowFactory.create(poifs);
             assertNotNull(ss);
-            npoifs.close();
+            poifs.close();
             assertCloseDoesNotModifyFile(file, ss);
         }
         // from OPCPackage
@@ -92,15 +92,15 @@ public abstract class BaseTestSlideShowFactory {
     @SuppressWarnings("resource")
     protected static void testFactoryFromProtectedNative(String protectedFile, String password) throws Exception {
         SlideShow<?,?> ss;
-        // Encryption layer is a BIFF8 binary format that can be read by NPOIFSFileSystem,
+        // Encryption layer is a BIFF8 binary format that can be read by POIFSFileSystem,
         // used for both HSLF and XSLF
 
-        // from protected NPOIFS
+        // from protected POIFS
         if (protectedFile.endsWith(".ppt") || protectedFile.endsWith(".pptx")) {
-            POIFSFileSystem npoifs = new POIFSFileSystem(fromFile(protectedFile));
-            ss = SlideShowFactory.create(npoifs, password);
+            POIFSFileSystem poifs = new POIFSFileSystem(fromFile(protectedFile));
+            ss = SlideShowFactory.create(poifs, password);
             assertNotNull(ss);
-            npoifs.close();
+            poifs.close();
             assertCloseDoesNotModifyFile(protectedFile, ss);
         }
         else {