Object not moving with code

So I thought I had it working, but now if I aim and switch off the gun then switch back it stays in the aiming position permanently. I have no idea why. here's the code that makes you aim

         if (Input.GetButtonDown("Fire2") && IsReloadingScript.IsReloading == false)
         {
             Crosshair.SetActive(false);
             transform.localPosition = AimDownSight;
             fpsCamP1.fieldOfView = AimZoom;
             
         }
 
         if (Input.GetButtonUp("Fire2"))
         {
             Crosshair.SetActive(true);
             transform.localPosition = HipFire;
             fpsCamP1.fieldOfView = 66;
         }

 

Heres the rest of the code

 using System.Collections;
 using System.Collections.Generic;
 using UnityEngine;
 using UnityEngine.UI;
 
 public class Gun : MonoBehaviour {
 
   
    
 
     public float damage = 10f;
     public float range = 100f;
     public float fireRate = 15f;
     public float impactForce = 30f;
     public float Ammo = 48f;
     public float clip = 6f;
     public float ReloadAmount = 6f;
     public float ReloadingTime = 1f;
     public float MovmentSlowWhileAiming;
     public float AimZoom = 60f;
     public string ReloadAnimationName;
 
     public AudioSource audioSource;
     public AudioClip RevolverShoot;
     public float volume = 0.5f;
 
     public Camera fpsCamP1;
     public ParticleSystem muzzleFlash;
     public GameObject impactEffectStone;
     public GameObject impactEffectDirt;
     public GameObject impactEffectWood;
     public GameObject impactEffectBlood;
     public GameObject DamagePopUp;
     public Text AmmoText;
     public Text CylinderText;
 
     private float nextTimeToFire = 0f;
     private WeaponSwitcher IsReloadingScript;
 
     public Vector3 AimDownSight;
     public Vector3 HipFire;
 
     public GameObject Crosshair;
     public GameObject WeaponHolder;
 
     public Animator anime;
     public Animator popUpTextAnime;
 
     private void OnEnable()
     {
         transform.localPosition = HipFire;
         Crosshair.SetActive(true);
         AmmoText.text = "Ammo " + Ammo.ToString();
         CylinderText.text = "Cylinder " + clip.ToString();
         fpsCamP1.fieldOfView = 66;
 
     }
 
     private void Start()
     {
         IsReloadingScript = WeaponHolder.GetComponent<WeaponSwitcher>();
         AmmoText.text = "Ammo " + Ammo.ToString();
         CylinderText.text = "Cylinder " + clip.ToString();
         transform.localPosition = HipFire;
     }
     // Update is called once per frame
     void Update () {
 
         
 
         //aiming
         if (Input.GetButtonDown("Fire2") && IsReloadingScript.IsReloading == false)
         {
             Crosshair.SetActive(false);
             transform.localPosition = AimDownSight;
             fpsCamP1.fieldOfView = AimZoom;
             
         }
         if (Input.GetButtonUp("Fire2"))
         {
             Crosshair.SetActive(true);
             transform.localPosition = HipFire;
             fpsCamP1.fieldOfView = 66;
         }
 
         //fire
         if (Input.GetButtonDown("Fire1") && Time.time >= nextTimeToFire && clip != 0 && IsReloadingScript.IsReloading == false)
         {
             audioSource.PlayOneShot(RevolverShoot, volume);
             nextTimeToFire = Time.time + 1f / fireRate;
             Shoot();
             CylinderText.text = "Cylinder " + clip.ToString();
 
 
         }
 
         //reload
         if (Input.GetKeyDown(KeyCode.R) && Ammo != 0 && Ammo > 0 && ReloadAmount != clip && IsReloadingScript.IsReloading == false)
         {
 
             Reloading();
 
     
 
         }
     }
 
 
  
 
 
   
 
     void Shoot()
     {
         muzzleFlash.Play();
         clip -= 1f;
 
         //Sends out an invisable ray and sees if it hits somtehing, then stores that in Hit.
 
         RaycastHit hit;
         if (Physics.Raycast(fpsCamP1.transform.position, fpsCamP1.transform.forward, out hit, range))
         {
            
             //test if the ray hits an object with the target code
             Target target = hit.transform.GetComponent<Target>();
             if (target != null)
             {
                 target.TakeDamage(damage);
                 GameObject DamageTextGameObject = Instantiate(DamagePopUp, hit.transform.position, fpsCamP1.transform.rotation);
                 Destroy(DamageTextGameObject, 1f);
              
             }
 
             //adds knockback to the object
             if (hit.rigidbody != null)
             {
                 hit.rigidbody.AddForce(-hit.normal * impactForce);
 
             }
 
             //Tests if hits stone. Shows stone hole effect
             if (hit.transform.tag.Contains("Stone"))
             {
                 GameObject impactGameObject = Instantiate(impactEffectStone, hit.point, Quaternion.LookRotation(hit.normal));
                 Destroy(impactGameObject, 2f);
 
             }
             //Tests if dirt/sand. shows dirt/sand hole effect
             if (hit.transform.tag.Contains("Dirt") || hit.transform.tag.Contains("Sand"))
             {
 
                 GameObject impactGameObject = Instantiate(impactEffectDirt, hit.point, Quaternion.LookRotation(hit.normal));
                 Destroy(impactGameObject, 2f);
 
             }
             //Tests if human. shows blood effect
             if (hit.transform.tag.Contains("Fleshy"))
             {
 
                 GameObject impactGameObject = Instantiate(impactEffectBlood, hit.point, Quaternion.LookRotation(hit.normal));
                 Destroy(impactGameObject, 2f);
 
             }
 
             //Tests if wood, shows wood effect
             if (hit.transform.tag.Contains("Wood"))
             {
 
                 GameObject impactGameObject = Instantiate(impactEffectWood, hit.point, Quaternion.LookRotation(hit.normal));
                 Destroy(impactGameObject, 2f);
 
             }
 
 
             
 
         }
 
    
 
     }
 
 
     private void Reloading()
     {
         StartCoroutine(Reload);
 
     }
 
     private IEnumerator Reload
     {
         get
         {
 
             for (int i = 0; clip < ReloadAmount; i++)
             {
                 IsReloadingScript.IsReloading = true;
                 
                 Ammo -= 1f;
                 clip += 1f;
                 anime.Play(ReloadAnimationName);
                 AmmoText.text = "Ammo " + Ammo.ToString();
                 CylinderText.text = "Cylinder " + clip.ToString();
                 yield return new WaitForSeconds(ReloadingTime);
 
             }
 
             IsReloadingScript.IsReloading = false;
 
 
 
         }
         
 
     }
     
 }