Hi Mark,

Yes absolute, makes great sense!

Thanks for the quick reply.

Jelmer

On Thursday, 10 April 2014 13:37:19 UTC+2, euromark wrote:
>
> Well, it is still a very valid and useful information.
> Most devs "look ahead" - and even if it is totally OK to use in 2.x, a 
> smart dev wouldn't do that for a new code/project.
> Why using deprecated functionality when you know it is going to be removed 
> in the future. When you know what the new "cleaner" method and approach for 
> it is?
>
> A smart dev uses the new functionality right away - so if anyone ever has 
> to upgrade that person doesn't have to get annoyed at himself or the 
> previous delevoper and his missing foresight.
> That saves time and money in the long run.
>
> The same goes for all functionality in 2.x that is outdated:
> - named params (=> query strins)
> - Set (=> Hash)
> - MediaView (response object)
> - ...
>
> One should even upgrade inside 2.x if possible as the new functionality 
> often times has improvements and less bugs in it as well.
>
> My five cents
> Mark
>
>
> Am Donnerstag, 10. April 2014 09:56:46 UTC+2 schrieb JelmerD:
>>
>> Hi guys,
>>
>> I am having some concerns on the current use of the @deprecated tag in 
>> the docblocks of several Cake 2.x methods.
>>
>> As the current Controller::disableCache() method states, it is deprecated 
>> (http://api.cakephp.org/2.4/source-class-Controller.html#978-987) but 
>> that is not true. It is still a valid method in CakePHP 2.x and as the 
>> comment on this tag states: *it will be removed in 3.0. *So, as long as 
>> you are developing in 2.x, this will never become a problem. It is "going 
>> to" become a problem, but only *if* you update to 3.x which I guess is 
>> not going to be the preferred option for every Cake developer that has a 
>> zillion lines of code.
>>
>> The reason I am having my concerns on the use of the tag, is that is 
>> misleading. Because, when I use the Controller::disableCache() method in my 
>> project, it is going to strike through the method and show a warning that 
>> it is deprecated, but after inspection of the docblock, it is not and it 
>> never will be in the 2.x releases. At least, that is what the comment 
>> states.
>>
>> I sincerely hope you understand my point of view on this subject and I 
>> would love to get a discussion started, because as far as I'm concerned, 
>> it's just throwing warnings that aren't really there and thus are messing 
>> up the view on the code.
>>
>> This issue exists for ~70 methods/functions in the current CakePHP 
>> library (source: find all "@deprecated" and scan globally for anything that 
>> says "3.0" in lib/Cake/)
>>
>

-- 
Like Us on FaceBook https://www.facebook.com/CakePHP
Find us on Twitter http://twitter.com/CakePHP

--- 
You received this message because you are subscribed to the Google Groups 
"CakePHP" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/cake-php.
For more options, visit https://groups.google.com/d/optout.

Reply via email to