Issue 177

[Geolim4]

Global changes:

• Added phpDoc Block comment on class/method/properties
• Replaced var keyword with public
• Defined non-existing visibility (public, protected, private)
• Moved constructor to be first method position
• Commented unused/emty if/else conditions
• Improved return consistency, e.g: phpfastcache_xcache::driver_isExisting()
• Removed aliases:
  • is_writeable() => is_writable()
• Removed tab indentations (it hurt eyes so bad)
• Removed silent operator (as per DeleteMulti() is not working properly (Not as defined in readme) #175):
  • sqlite.php
  • file_exists()
  • mkdir()
• Ignored:
  • phpfastcache/_extensions/predis-1.0/*
  • phpfastcache/_extensions/SSDB.php

[Geolim4]

Warning, i resolved a conflict on some files, can you check if a don't forget anything during the merge ?
Thanks.

• Conflicts:
  • phpfastcache/3.0.0/abstract.php
  • phpfastcache/3.0.0/drivers/files.php
  • phpfastcache/3.0.0/drivers/sqlite.php
  • phpfastcache/3.0.0/phpfastcache.php

[Geolim4]

Related issue: #177

[Geolim4]

Already merged ? Oo
So fast !

[khoaofgod]

I trust you, hahahaha

On Mon, Feb 1, 2016 at 5:52 PM, Georges notifications@github.com wrote:

Already merged ? Oo
So fast !

—
Reply to this email directly or view it on GitHub
https://github.com/khoaofgod/phpfastcache/pull/181#issuecomment-178307716
.

[khoaofgod]

it merged, but it's not a release yet. ^_^

On Mon, Feb 1, 2016 at 5:52 PM, Banh Mi DSLR101 khoaofgod@gmail.com wrote:

I trust you, hahahaha

On Mon, Feb 1, 2016 at 5:52 PM, Georges notifications@github.com wrote:

Already merged ? Oo
So fast !

—
Reply to this email directly or view it on GitHub
https://github.com/khoaofgod/phpfastcache/pull/181#issuecomment-178307716
.

[Geolim4]

Ok :)

[khoaofgod]

I will test it after work today or tmr before I draft a new release on this commit.
Thanks again for help me on this, I was overload from last summer.

[Geolim4]

You're welcome, i love readable and well-commented code :))