flowjs / flow-php-server

flow.js php server library, validates uploaded chunks and safely merges all chunks to a single file
http://flowjs.github.io/ng-flow/
MIT License
245 stars 42 forks source link

PHP5.4 missing abstract function wgetCurrentChunkNumber #2

Closed bensquire closed 10 years ago

bensquire commented 10 years ago

I'm using PHP 5.4 and when using the Flow\Request object I get the error that the abstract class method 'wgetCurrentChunkNumber' must be declared. Which it isn't... I'd submit a fix, but I'm not even sure what the function would do...

Cheers Ben

AidasK commented 10 years ago

I have tested this library on 5.4 php and it is working all good. Have you installed it with composer? Maybe library was edited?

Method should be called getCurrentChunkNumber. https://github.com/flowjs/flow-php-server/blob/master/src/Flow/Request.php#L84 https://github.com/flowjs/flow-php-server/blob/master/src/Flow/RequestInterface.php#L46

Closing this for now.

bensquire commented 10 years ago

I did install using composer, yes:

"flowjs/flow-php-server": "dev-master"

However I must have accidently inserted the 'w'... apolgies!