#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 harking):
I've attached a small patch that performs a check to make sure a database
session is not being used before calling unlink. It also removes the @
before the unlink calls.
There is an issue with this patch in that they might be using another type
of session besides 'database' that still is using another means of storing
the session (memcache) an this will throw an error still.
There is also another @session_destroy() error suppression but phpnut
believed it needed to be in there: https://trac.cakephp.org/changeset/1953
I'm thinking it is if destroy() is called before the session is ever
started. We may want to throw an error in this case?
--
Ticket URL: <https://trac.cakephp.org/ticket/3877#comment:12>
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
-~----------~----~----~----~------~----~------~--~---
