Need Help about identifier npc!

[Limestone]

Everytime i talk to the npc, im getting 1 only identified items. is there anything wrong with this script?

 

 

                            mes "[ ^ffa500Eternal Kafra^000000 ]";
                            mes "Need me to identify your equipment?";
                            next;
                            prompt("Yes, please.:No thanks.");
                            switch(@menu){
                            case 1:
                                    mes "[ ^ffa500Eternal Kafra^000000 ]";
                                    mes "Okay, let me have a look at your inventory.";
                                    next;
                                    getinventorylist;
                                    while( .@i < @inventorylist_count ){
                                        if ( !@inventorylist_identify[.@i] ){
                                            delitem2 @inventorylist_id[.@i],1,0,0,0,0,0,0,0;
                                            getitem @inventorylist_id[.@i],1;
                                            mes "[ ^ffa500Eternal Kafra^000000 ]";
                                            mes "I have successfully identified "+.@i+" items.";
                                            next;
                                            goto Kaf_End;
                                        }
                                        .@i++;
                                    }
                                    mes "[ ^ffa500Eternal Kafra^000000 ]";
                                    mes "Everything has already been identified.";
                                    next;
                                    goto Kaf_End;
                                    end;

Edited September 19, 2013 by Capuche
Edit the title

[Patskie]

Replace the identify part by this : 

case 1:
        mes "[ ^ffa500Eternal Kafra^000000 ]";
        mes "Okay, let me have a look at your inventory.";
        next;
        getinventorylist;
            while( .@idn < @inventorylist_count ){
                if ( !@inventorylist_identify[.@idn] ){
                    delitem2 @inventorylist_id[.@idn],1,0,0,0,0,0,0,0;
                    getitem @inventorylist_id[.@idn],1;
                }
                .@idn++;
            }
        mes "[ ^ffa500Eternal Kafra^000000 ]";
        mes "Everything has already been identified.";
        next;
        goto Kaf_End;
        end;

[Patskie]

Can you show the whole script?

[Limestone]

Can you show the whole script?

 

http://pastebin.com/raw.php?i=neqSApyp

[Limestone]

where should i put this dialog? if there's a unidentified items, it will show this "I have successfully identified "+.@idn+" items." but if there's no unidentified items, it will show this, "Everything has already been identified."

@EDIT: FIXED.

[nanakiwurtz]

You set your own post as the best answer, funny lol