In rebuildfstab, you don't want the $ in front of Israel, as you are now using the explicit string and not a string parameter label. So your lines should read:
Code:
  [ -n "$user" ] && options="$options,uid=Israel" 
   [ -n "$group" ] && options="$options,gid=Israel"
[Edited 09:07hrs GMT]:-
In /etc/fstab you still(or again) have the commas which are now trailing the end of umask=000 in
/dev/hda1 /media/hda1 ntfs noauto,users,exec,umask=000, 0 0
and at the end of shortname=winnt in
/dev/sda1 /media/sda1 vfat noauto,users,exec,umask=000,shortname=winnt, 0 0
(They may have got back in there as a consequence of the rebuildfstab failing on uid=$Israel and gid=$Israel)

Edit the above and see how that works - good luck