Yup, that is it ( sequence is not an issue ) as long as you don't have any other security policies and the syslog configuration is the same it is correct ( all security policies have to have the " then log" ).
If it is not working double check your syslog server and make sure that there is traffic traversing the SRX in the first place.