Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecation cleanup #951

Merged
merged 52 commits into from Nov 15, 2012
Merged

Deprecation cleanup #951

merged 52 commits into from Nov 15, 2012

Conversation

halkyon
Copy link
Contributor

@halkyon halkyon commented Nov 15, 2012

Removing all the deprecated 2.4/2.5/3.0 things.

CMS pull request: silverstripe/silverstripe-cms#239 (please pull that one after this)

Unit tests will probably fail as a result, since the CMS pull request doesn't get tested on Travis with the dependent framework changes here.

Sean Harvey added 30 commits November 15, 2012 14:43
Use the methods param() and params() on SS_HTTPRequest instead.
Director::redirect() -> Controller->redirect()
Director::redirect_to() -> Controller->redirectedTo()
Director::set_status_code() -> Controller->getResponse()->setStatusCode()
Director::get_status_code() -> Controller->getResponse()->getStatusCode()
Director::redirectBack() -> Controller->redirectBack()
Use SS_ENVIRONMENT_TYPE in your _ss_environment.php file, or
Director::set_environment_type()
Use the individual getters on PaginatedList instead.
Use FormAction with setAttribute('src', 'myimage.png') and custom
JavaScript to achieve hover effect.
Use UploadField with
a$myField->getValidator()->setAllowedExtensions(array('jpg', 'gif', 'png'))
Use ArrayData::toMap() instead
Use $member->addToGroupByCode($groupcode) instead
Use PasswordEncryptor::check() instead
Use Zend_Translate instead.
Use LowerCase() and UpperCase() instead
Use custom Javascript validation instead
Sean Harvey added 21 commits November 15, 2012 14:43
Use setRightTitle() and setFolderName() instead
Use FieldList API through Form::Fields() and Form::Actions() instead
Use SecurityToken class directly instead
Use leftJoin() or innerJoin() instead
Use 2 arguments, the first being limit and the second offset instead.
This is not used internally any longer
…elds()

Use database_fields() and custom_database_fields() instead
setTabIndex() -> setAttribute('tabindex')
getTabIndex() -> getAttribute('tabindex')
describe() -> setDescription()
Use setColumns() and setRows() instead
Use ToggleCompositeField->setStartClosed() instead
GridField uses createTag() which is marked for deprecation, rather
than have it used as the cornerstone of generating FormField templates,
use it as a helper in case fields generate HTML tags from PHP.
stojg pushed a commit that referenced this pull request Nov 15, 2012
@stojg stojg merged commit 49e54c8 into silverstripe:master Nov 15, 2012
@chillu
Copy link
Member

chillu commented Nov 15, 2012

Legend!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants