Need help in code

Note, it's helpful to use the forums CODE tags. It's hard to read code with no indentation.

Just a few notes:

You don't say if nothing at all shows up on the LED's or if it is just wrong. Most boards turn an LED on when the value is '0'. So you may want to use:

led <= not data;

Have you tried just driving "led" with a fixed bit pattern to make sure everything is hooked up and functioning? When developing code, it is helpful to do it a small step at a time and make sure things work at each step.

Do you know the value of "switch"? With Quartus, SignalTap would probably show your issue very quickly. Xilinx has ChipScope, but, unless something has changed, it is a pain to use.

Your clk_2G is more of a pulse. Have you checked that your tools are interpreting it as a clock? You set it to 1 for one clock cycle of clk every 25,000,000 cycles. A more common way to do this would be:

process(clk)
begin
    if clk'event and clk='1' then
        if tooluur = 12_500_000 then --2Hz   -- I'm assuming clk is at 50 Mhz. 
            tooluur <= 1;                               -- You're not at exactly 2 Hz. Minor in this case. 
            clk_2G <= not clk_2G;                  -- Just invert each time. 
       else
           tooluur <= tooluur+1;
       end if;
    end if;
end process;

However, I would not use your clk_2G at all. I'd get rid of it and replace it with a pulse say "check_button". Set check_button to '1' every time tooluur gets to 25,000,000 (in other words, exactly what you were doing). Then in the first process, now clocked by 'clk', put the body of the process inside "if check_button = '1' then". This assumes you really only want to check the button state twice a second as opposed to continuously. Clocks and clock nets are special and scarce resources. Don't use them unless you really need to.

You're only checking "button" twice a second. I assume it is pressed and held?

Asynchronous inputs, which I assume "button" is, should have a few synchronization stages to avoid metastability issues. And it should be debounced (but probably not with a 2 Hz clock).

You could use VHDL "rol" operator on "data" instead of "temp".

I notice you're using std_logic_arith instead of numeric_std. Is there a reason for that?

It's best to replace magic numbers with constants.

As others have mentioned, you should initialize all signals and variables. For synthesis, uninitialized values are probably zero. But simulators will complain.

And yes, I had nothing else to do today.