CoffeeScript if statement erroring out in Rails

Question

For some reason, I'm getting SyntaxError: unexpected POST_IF somewhere in this code. Any ideas? -

$('.provider-fields').hide()
  switch parseInt($('#referral_provider_type').val())
    when window.AIDIN.provider_types.hha
      $('.hha-fields').show()
    when window.AIDIN.provider_types.snf
      $('.snf-fields').show()
      $("#length_of_stay").show()
      if ( $("#regular").hasClass('active') )
        $("#search_radius").show()
        if ( $("#search_radius input").val() == '' )
          $("#search_radius input").val window.AIDIN.radii.snf_radius
    when window.AIDIN.provider_types.ltac
      $('.ltac-fields').show()
      if ( $("#regular").hasClass('active') )
        $("#search_radius").show()
        if ( $("#search_radius input").val() == '' )
        $("#search_radius input").val window.AIDIN.radii.ltac_radius
    when window.AIDIN.provider_types.pd
      $('.pd-fields').show()
    when window.AIDIN.provider_types.irf
      $('.irf-fields').show()
      if ( $("#regular").hasClass('active') )
        $("#search_radius").show()
        if ( $("#search_radius input").val() == '' )
        $("#search_radius input").val window.AIDIN.radii.irf_radius

  $('#referral_provider_type').live 'change', (e) ->
    $('.provider-fields').hide()
    $('input[type="checkbox"][name*="referral[referral_care_types_attributes]"]').attr("checked", false)
    switch parseInt($('#referral_provider_type').val())
      when window.AIDIN.provider_types.hha
        $('.hha-fields').show()
      when window.AIDIN.provider_types.snf
        $('.snf-fields').show()
        $("#length_of_stay").show()
        if ( $("#regular").hasClass('active') )
          $("#search_radius").show()
          if ( $("#search_radius input").val() == '' ) 
          $("#search_radius input").val window.AIDIN.radii.snf_radius
      when window.AIDIN.provider_types.ltac
        $('.ltac-fields').show()
        if $( ("#regular").hasClass('active') )
          $("#search_radius").show()
          if ( $("#search_radius input").val() == '' )
          $("#search_radius input").val window.AIDIN.radii.ltac_radius
      when window.AIDIN.provider_types.pd
        $('.pd-fields').show()
      when window.AIDIN.provider_types.irf
        $('.irf-fields').show()
        if ( $("#regular").hasClass('active')   )
          $("#search_radius").show()
          if ( $("#search_radius input").val() == '' )
          $("#search_radius input").val window.AIDIN.radii.irf_radius

Rewrite this code? Haven't seen such a mess for a looong time (don't take it personally) — Mike Szyndel
– Mike Szyndel, Commented Jul 24, 2013 at 17:37
You don't need parentheses around the conditions in if statements. Just write if $("#regular").hasClass('active'). — wwww
– wwww, Commented Jul 24, 2013 at 17:39

vee · Accepted Answer · 2013-07-24 17:46:10Z

Indentation problem as shown below:

  switch parseInt($('#referral_provider_type').val())
    when window.AIDIN.provider_types.hha
      $('.hha-fields').show()
    when window.AIDIN.provider_types.snf
      $('.snf-fields').show()
      $("#length_of_stay").show()
      if ( $("#regular").hasClass('active') )
        $("#search_radius").show()
        if ( $("#search_radius input").val() == '' )
          $("#search_radius input").val window.AIDIN.radii.snf_radius
    when window.AIDIN.provider_types.ltac
      $('.ltac-fields').show()
      if ( $("#regular").hasClass('active') )
        $("#search_radius").show()
        if ( $("#search_radius input").val() == '' )
          $("#search_radius input").val window.AIDIN.radii.ltac_radius
    when window.AIDIN.provider_types.pd
      $('.pd-fields').show()
    when window.AIDIN.provider_types.irf
      $('.irf-fields').show()
      if ( $("#regular").hasClass('active') )
        $("#search_radius").show()
        if ( $("#search_radius input").val() == '' )
          $("#search_radius input").val window.AIDIN.radii.irf_radius

  $('#referral_provider_type').live 'change', (e) ->
    $('.provider-fields').hide()
    $('input[type="checkbox"][name*="referral[referral_care_types_attributes]"]').attr("checked", false)
    switch parseInt($('#referral_provider_type').val())
      when window.AIDIN.provider_types.hha
        $('.hha-fields').show()
      when window.AIDIN.provider_types.snf
        $('.snf-fields').show()
        $("#length_of_stay").show()
        if ( $("#regular").hasClass('active') )
          $("#search_radius").show()
          if ( $("#search_radius input").val() == '' ) 
            $("#search_radius input").val window.AIDIN.radii.snf_radius # <----- Here
      when window.AIDIN.provider_types.ltac
        $('.ltac-fields').show()
        if $( ("#regular").hasClass('active') )
          $("#search_radius").show()
          if ( $("#search_radius input").val() == '' )
            $("#search_radius input").val window.AIDIN.radii.ltac_radius # <-------Here
      when window.AIDIN.provider_types.pd
        $('.pd-fields').show()
      when window.AIDIN.provider_types.irf
        $('.irf-fields').show()
        if ( $("#regular").hasClass('active')   )
          $("#search_radius").show()
          if ( $("#search_radius input").val() == '' )
            $("#search_radius input").val window.AIDIN.radii.irf_radius # <---------Here

Collectives™ on Stack Overflow

CoffeeScript if statement erroring out in Rails

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related