Skip to content

Commit

Permalink
Merge pull request #249 from quetzyg/4.0
Browse files Browse the repository at this point in the history
[4.0.7] Fix #233 properly
  • Loading branch information
anteriovieira authored Jun 4, 2017
2 parents 6aaa175 + 9d063cf commit 6476d55
Show file tree
Hide file tree
Showing 4 changed files with 39 additions and 2 deletions.
1 change: 1 addition & 0 deletions .gitattributes
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
tests export-ignore
.gitattributes export-ignore
.github export-ignore
.gitignore export-ignore
.travis.yml export-ignore
CHANGELOG.md export-ignore
Expand Down
29 changes: 29 additions & 0 deletions .github/ISSUE_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
| Q | A
| ----------------- | ---
| Bug? | no|yes
| New Feature? | no|yes
| Framework | Laravel|Lumen
| Framework version | 5.x.y
| Package version | 4.x.y
| PHP version | 5.x.y|7.x.y

#### Actual Behaviour

Describe the behaviour you're experiencing.


#### Expected Behaviour

Describe the behaviour you're expecting.


#### Steps to Reproduce

List all the steps needed to reproduce the issue. Make sure to include the code you're using,
any screenshots and/or other resources you find pertinent.


#### Possible Solutions

If you have any ideas on how to solve the issue, add them here.
If not, omit this part.
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,10 @@
## v4.0.7 (2017-06-04)
### Added
- GitHub issue template file

### Fixed
- Properly fixed issue ([#233](https://github.com/owen-it/laravel-auditing/issues/233))

## v4.0.6 (2017-05-21)
### Fixed
- Calling a member function on null ([#244](https://github.com/owen-it/laravel-auditing/issues/244))
Expand Down
4 changes: 2 additions & 2 deletions src/AuditableObserver.php
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ public function created(AuditableContract $model)
public function updated(AuditableContract $model)
{
// Ignore the updated event when restoring
if (!static::$restoring || true) {
if (!static::$restoring) {
Auditor::execute($model->setAuditEvent('updated'));
}
}
Expand Down Expand Up @@ -92,7 +92,7 @@ public function restored(AuditableContract $model)
Auditor::execute($model->setAuditEvent('restored'));

// Once the model is restored, we need to put everything back
// as before, in case an legitimate update event is fired
// as before, in case a legitimate update event is fired
static::$restoring = false;
}
}

0 comments on commit 6476d55

Please sign in to comment.