#3877: @ should not be used to suppress notices
--------------------------------+-------------------------------------------
Reporter: mensler | Owner:
Type: Optimization | Status: new
Priority: Low | Milestone: 1.2.x.x
Component: General | Version: RC1
Severity: Minor | Resolution:
Keywords: | Php_version: PHP 5
Cake_version: 1.2.0.6311 beta |
--------------------------------+-------------------------------------------
Comment (by mensler):
Replying to [comment:6 nate]:
> Not smart. An extra stat call is slower than error suppression.
I've tested this. {{{file_exists($file) && unlink($file);}}} is 6 times
faster than [EMAIL PROTECTED]($file);}}}.
So I still would recommend to change this in CakeSession::destroy() and
CakeSession::!__regenerateId() or at least check if Session.save is not
'database'.
--
Ticket URL: <https://trac.cakephp.org/ticket/3877#comment:11>
CakePHP : The Rapid Development Framework for PHP <https://trac.cakephp.org/>
Cake is a rapid development framework for PHP which uses commonly known design
patterns like ActiveRecord, Association Data Mapping, Front Controller and MVC.
Our primary goal is to provide a structured framework that enables PHP users at
all levels to rapidly develop robust web applications, without any loss to
flexibility.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"tickets cakephp" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/tickets-cakephp?hl=en
-~----------~----~----~----~------~----~------~--~---
