TWo PIC 18f4620 , one as master & the second is slave I2C communication problem

I use "Mikroc for PIC pro 6.0" and i write a program to communicate between  two 18f4620 one is master and the other is slave
 
MASTER CODE 
 

unsigned short int sendData=0;
//######################################################################################################################################
void main() 
{
 OSCCON.SCS1=1;
 OSCCON.SCS0=0; // external oscillator HS oscillator option 4MHZ 
//from "edit project" option in mikroc program
 CMCON = 0b00000111;
 TRISC.B3 = 1;
 TRISC.B4 = 1; 
 I2C1_Init(100000); //Inicializar Bus I2C a 100kHz
 delay_ms(500);
 while(1)
  {
           // initialize I2C communication
   I2C1_Start(); // issue I2C start signal
   I2C1_Wr(0x18); // send byte via I2C (device address + W)
   I2C1_Wr(4); // send byte (address of EEPROM location)
   I2C1_Wr(0xAA); // send data (data to be written)
   I2C1_Stop();
   delay_ms(2000);
  }
}

 
Slave code

  int counter=0,address=0,rxbuffer=0,buff=0, new=0;

 void interrupt()
  {
    if (PIR1.SSPIF)
    { 
      new=1; 
      SSPCON1.CKP = 0; 
       if(!SSPSTAT.B2 & SSPSTAT.B5 & SSPSTAT.B0 )
        {
            counter++;
            if (counter==1)
             {
               address=SSPBUF; 
             } 

            if (counter==2)
             {
               rxbuffer=SSPBUF; 
               counter=0;
             }
            PIR1.SSPIF = 0; 
            SSPCON1.CKP=1;
            new=1;
        }
       if( !SSPSTAT.B2 & !SSPSTAT.B5 & SSPSTAT.B0 )
        {
            buff = SSPBUF;
            PIR1.SSPIF = 0;
            SSPCON1.CKP=1;
            new=1;
        }
       PIR1.SSPIF = 0;
     }
  }
void main() 
{
  ADCON1=0b00001111;
  CMCON=0b00000111;
  TRISC.B3=1; 
  TRISC.B4=1; 
  INTCON=0b00000000;
  SSPCON1=0b00110110; 
  SSPCON2=0b00000001;
  SSPADD=0x18; 
  SSPSTAT=0x00;
  SSPCON1.SSPEN=1;
  SSPSTAT.SMP=1;
  SSPSTAT.CKE=0;
  PIR1.SSPIF=0;
  PIR2.BCLIF=0;
  PIE1.SSPIE=1; 
  INTCON.PEIE=1; 
  INTCON.GIE=1; 
  TRISA.B0=0;
  while(1)
  {
     if(new==1)
     {
     PORTA.B0=1;
     new=0;
     }
  }

}

 
 
i use this combination in proteus for testing so i should fin d the led on portA pin0 is lighting 
which is never happen so the  PIR1.SSPIF never gets one so we can say that the slave doesnt give a match for her calling address from the master . The one million dollar question  is " Why doesnt she generate an address matching interrupt to make "PIR1.SSPIF" equal 1? 

HERE U ARE PICTURES ABOUT THE COMBINATION OF THE TWO PICS THAT MAY HELP U TO HELP ME 
 
link for first image  https://imgur.com/a/UpyZY

link for second image https://imgur.com/1pYF0d6 

HERE U ARE  ANOTHER PICTURES ABOUT THE COMBINATION OF THE TWO PICS THAT MAY HELP U TO HELP ME . fULL WAVE ON THE OSCILLOSCOPE 
 
link for first image https://imgur.com/yBWFD9x

+1
or the other way round. If the slave defines

SSPADD=0x18; 

then try

I2C1_Wr(0x18<<1); // send byte via I2C (device address + W)

for the master to add the least significant bit for R/W
 
[edit: as it turns out two days later, here I was wrong unfortunately. An i2c slave on a PIC24 and a PIC18 use a different address scheme. My explanations are valid for the master and a PIC24 slave. But the PIC18 expects the fully address byte and simply ignores the least significant bit (R/W).  E.g. here it would for an i2c address 0x18, the master sends 0x30 for a WRITE, the slaves sets 0x30 in SPADD and will simply not use the LSB. ]

rodims
+1
or the other way round. If the slave defines

SSPADD=0x18; 

then try

I2C1_Wr(0x18<<1); // send byte via I2C (device address + W)

for the master to add the least significant bit for R/W

 Why should i shift to the left ?
the address the 8th BIT is already zero for write 

rodims
The shift left just makes clear that you make room for the R/W bit. I would not code it like that of course.
The 7 MSB are the device address, the LSB is the R/W.
Thus your device master currently uses a device address of 0x0c. And a LSB '0' for WRITE.
If the addresses don't match, there will be no interrupt.
Why don't you just try ?
 
https://support.saleae.co...ter-Integrated-Circuit
 

I dont understand what u want me exactly to write and where  in slave code or in master code
beside i dont get what u say about shifted and un-shifteted so i will be grateful if u enlighten me more pls 

rodims
The shift left just makes clear that you make room for the R/W bit. I would not code it like that of course.
The 7 MSB are the device address, the LSB is the R/W.
Thus your device master currently uses a device address of 0x0c. And a LSB '0' for WRITE.
If the addresses don't match, there will be no interrupt.
Why don't you just try ?
 
https://support.saleae.co...ter-Integrated-Circuit
 

BUT 
if u want me to write in  master code
 
 

I2C1_Wr(0x18<<1); // send byte via I2C (device address + W)

 
and to write in slave code 
 

SSPADD=0x0C; 

 
 
i did it and i dont understand why but it didnt work the same no address match

rodims
No, I did not want you to do that. Now you mangled both sides.  (writing 0x30 in slave, using ADDR 0x0c in slave)
 
Please use 0x18 in your slave (as you originally did).
And use 0x30 in the master for WRITE and 0x31 for READ.
 
[edit: I did not see that you already added multiple posts. If you already tested my above proposal, then you need to search for additional problems]

Master

 
while(1)
 {
 // initialize I2C communication
 I2C1_Start(); // issue I2C start signal
 I2C1_Wr(0x30); // send byte via I2C (device address + W)
 I2C1_Wr(4); // send byte (address of EEPROM location)
 I2C1_Wr(0xAA); // send data (data to be written)
 I2C1_Stop();
 delay_ms(2000);
 //WriteToSlavePic(4,sendData);
 sendData++;
 // Delay_1sec();
 }
}
 

 
slave
 

 
INTCON=0b00000000;
 SSPCON1=0b00110110; // Modo esclavo 7 bits de direccion.
 SSPCON2=0b00000001;
 SSPADD=0x18; //Dirección del dispositivo
 SSPSTAT=0x00;
 SSPCON1.SSPEN=1;
 SSPSTAT.SMP=1;
 SSPSTAT.CKE=0;
 PIR1.SSPIF=0;
 PIR2.BCLIF=0;
 PIE1.SSPIE=1; // Habilita la interrupcion del modulo SSP
 INTCON.PEIE=1; // Habilita las interrupciones de los perifericos
 INTCON.GIE=1;
 

 
I tried as u see 
but no  change 
 
i want ur help to fin d out what is the "more problem" do i have because it seems for me that there is no but i dont know what to do more to make it work 

And of course you should apply Darios changes from #2 (&&), and then place a corrected version of the complete master and slave code here ...
 

I think there is something with the clock and data sent and received  . i just feel that because simply the

new =0;

 is out of the

!SSPSTAT.B2==1 && SSPSTAT.B5==1 && SSPSTAT.B0==1

regardless is it '&' or '&&' as i guess in the above version of the code it should be & not && be cause it is logically ANDIng the three bits if the result is 1 so the if condition is true
anyway the

new

variable is out of the if condition so once the interrupt happens , portA.B0 should light the LED up