Skip to content

Commit

Permalink
Resolve some PHP 8 deprecations (#30)
Browse files Browse the repository at this point in the history
Co-authored-by: Nick Huang <[email protected]>
  • Loading branch information
sanmai and nick322 authored Jun 11, 2023
1 parent 1eb77c8 commit 6bb4365
Show file tree
Hide file tree
Showing 5 changed files with 26 additions and 1 deletion.
6 changes: 6 additions & 0 deletions OLE.php
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,12 @@ class OLE extends PEAR
*/
var $bigBlockSize;

/**
* The big block threshhold.
* @var int
*/
var $bigBlockThreshold;

/**
* Size of small blocks. This is usually 64.
* @var int number of octets per block
Expand Down
2 changes: 2 additions & 0 deletions OLE/ChainedBlockStream.php
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,8 @@ class OLE_ChainedBlockStream extends PEAR
*/
var $pos;

var $context;

/**
* Implements support for fopen().
* For creating streams using this wrapper, use OLE_PPS_File::getStream().
Expand Down
8 changes: 8 additions & 0 deletions OLE/PPS/File.php
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,14 @@ class OLE_PPS_File extends OLE_PPS
*/
var $_tmp_dir;

/**
* The temporary file handle
* @var resource
*/
var $_PPS_FILE;

var $_tmp_filename;

/**
* The constructor
*
Expand Down
10 changes: 9 additions & 1 deletion OLE/PPS/Root.php
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,15 @@ class OLE_PPS_Root extends OLE_PPS
* @var string
*/
var $_tmp_dir;


var $_BIG_BLOCK_SIZE;

var $_SMALL_BLOCK_SIZE;

var $_tmp_filename;

var $_FILEH_;

/**
* Constructor
*
Expand Down
1 change: 1 addition & 0 deletions phpunit.xml.dist
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
<?xml version="1.0" encoding="UTF-8"?>
<phpunit bootstrap="vendor/autoload.php" verbose="true" colors="true"
convertNoticesToExceptions="true"
convertDeprecationsToExceptions="true"
>
<testsuites>
<testsuite>
Expand Down

0 comments on commit 6bb4365

Please sign in to comment.