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.

Sign up for GitHub

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

Jump to bottom

DBField Documentation correction #9531

Merged

emteknetnz merged 1 commit into silverstripe:4 from alessandromarotta:patch-1

Jun 30, 2020

alessandromarotta commented Jun 2, 2020

Hello,
the method requiredField in the line 40 of the API documentation doesn't exist in the class DB.

I think that it should be DB::require_field($this->tableName, $this->name, "blob");


          Documentation correction

fa406c1

emteknetnz approved these changes

View reviewed changes

Member

emteknetnz left a comment

Yes that's correct - https://github.com/silverstripe/silverstripe-framework/blob/4/src/ORM/DB.php#L597

Thanks for the pull-request, I'll merge this

emteknetnz merged commit 16914bf into silverstripe:4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment