Yes, that job as-done is incorrect, though I could see where a novice working with poor instructions might think it is correct.
You need one of the types of boxes mentioned by Jack or Machavity.
In particular, note how you have the Romex or UF coming through the oval holes. I get where that makes sense. However actually those oval holes are variable screw-mount holes, for fixture mounting studs (just like the ones currently in use, but with a narrower range of spacing). They don't have grommets or edge protection so they would eventually chew into the cable sheath.
You must enter the boxes with proper cable clamps, assuming the box doesn't have cable clamps built-in as many do. Note that Jack's box is made for conduit piping, with standard knockouts. You can get cable clamps that fit in a knockout, but they also make metal boxes with cable clamps built-in. Plastic boxes have only cable clamps and don't even have knockouts (since metal conduit grounds through the metal box).