Skip to main content
Code Review

Return to Question

Notice removed Draw attention by Razvan Zamfir
Bounty Ended with Sᴀᴍ Onᴇᴌᴀ's answer chosen by Razvan Zamfir
add tag
Link
deleted 10 characters in body
Source Link
public function deleteImage($id, $fileName) {
 $article = Article::find($id);
 $article->image = "default.jpg";
 $article->save();
 if (File::exists(public_path('images/articles/' . $fileName))) {
 File::delete(public_path('images/articles/' . $fileName));
 }
}
public function deleteImage($id, $fileName) {
 $article = Article::find($id);
 $article->image = "default.jpg";
 $article->save();
 if (File::exists(public_path('images/articles/' . $fileName))) {
 File::delete(public_path('images/articles/' . $fileName));
 }
}
public function deleteImage($id, $fileName) {
 $article = Article::find($id);
 $article->image = "default.jpg";
 $article->save();
 if (File::exists(public_path('images/articles/' . $fileName))) {
 File::delete(public_path('images/articles/' . $fileName));
 }
}
added 6 characters in body
Source Link
  1. Is there any redundancy in my code?
  2. Do you see any security issues?
  3. Do you see any usability or UX issues?
  1. Is there any redundancy in my code?
  2. Do you see any security issues?
  3. Do you see any usability issues?
  1. Is there any redundancy in my code?
  2. Do you see any security issues?
  3. Do you see any usability or UX issues?
Notice added Draw attention by Razvan Zamfir
Bounty Started worth 50 reputation by Razvan Zamfir
edited tags
Link
Loading
Loading
added 169 characters in body
Source Link
Loading
added 2 characters in body
Source Link
Loading
added 14 characters in body
Source Link
Loading
Source Link
Loading
lang-php

AltStyle によって変換されたページ (->オリジナル) /