Skip to content
This repository has been archived by the owner on Jul 16, 2023. It is now read-only.

Add saveWithoutHydrating method #224

Closed
wants to merge 1 commit into from
Closed

Add saveWithoutHydrating method #224

wants to merge 1 commit into from

Conversation

WarrenMoore
Copy link

Adds a $skipEntityHydration parameter to the validate method, and adds a saveWithoutHydrating method.

@igorsantos07
Copy link
Member

  1. try to not change line spacing. That makes it hard to understand what's being really changed. We'll fix code styling in another issue (Add CI for code style verifications #268)
  2. What's the point of the change? When you send a PR with a new feature, please elaborate a little bit on why is it needed, what's the use cases and such.

hhsadiq
hhsadiq reviewed Jun 2, 2015
View reviewed changes
src/LaravelBook/Ardent/Ardent.php
if ($this->forceEntityHydrationFromInput || (empty($this->attributes) && $this->autoHydrateEntityFromInput)) {
$customMessages = (empty($customMessages))? static::$customMessages : $customMessages;

$canHydrate = $this->forceEntityHydrationFromInput || (empty($this->attributes) && $this->autoHydrateEntityFromInput);
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There could be a better way to do this

@igorsantos07
Copy link
Member

Closing as OP never responded and there were no reactions to the PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
Review RFC
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@WarrenMoore @igorsantos07 @hhsadiq