Jump to content
  • 0

[HELP] Castle Control Announce During Siege


Question

Posted

Hello Devs of Maxcheaters,

 

I've came to this topic after a long-term period of trying to solve this problem.

I want when a Clan takes a Castle during siege,to be announced to all players.

Using the method that i've already used to announce when Siege starts,stops & when clan registers for Attack / Defend,i've tryed to use the same code.

 

This is the code:

Announcements.getInstance().announceToAll(""+player.getClan().getName()+" has taken "+getCastle().getName()+" Castle.");

 

This is the error:

errorck.jpg

 

But when using the same code for the other lines,it works perfect.

startg.jpg

endmu.jpg

registerzw.jpg

 

Any ideas?

 

Thanks in advance,

 

- Revo

Recommended Posts

  • 0
Posted

First you can use this code, it's just cleaner.

 

Announcements.getInstance().announceToAll(player.getClan().getName()+" has taken "+getCastle().getName()+" Castle.");

 

Second, your code can work only if the castle was to another clan. It can't work if the castle was to NPCs.

 

if (getCastle().getOwnerId() > 0) means "if the castle got an owner". You surely made your test on a siege where NPCs were defenders. Your code work, but only on sieges where there is a clan owner, or if someone retake the castle during the siege period (and if only the clan was stored, i'm not even sure it works like that).

 

        if (getIsInProgress()) // Siege still in progress
       {
           // Announce
           Announcements.getInstance().announceToAll(player.getClan().getName()+" has taken "+getCastle().getName()+" Castle.");

           if (getCastle().getOwnerId() > 0) 
           	_siegeGuardManager.removeMercs(); // Remove all merc entry from db

           // If defender doesn't exist (Pc vs Npc) and only 1 attacker

  • 0
Posted

Tryskell,

 

Thanks for your reply & for your cleaner code.

My point is that for the following parts of the siege,the code that i am using works successfully.

 

Siege Start

Siege End

Register Attacker / Defender

 

But when i am trying to make it work for Castle Control,it doesn't accept the code.

player.getClan().getName()

Seems to be invalid but i can't understand the reason.

player

 

The exact code works for all the other lines.

That's what confuses me the most!

 

Thanks in advance,

 

- Revo

  • 0
Posted

Show me a little bit more of this class, whole class (Even if its big, and i suppose it is) put it at pastebin and upload link here.

 

You can't "just create" object of L2PcInstance, because it will return null everywhere (means nothing + NPE error in addition xD)

  • 0
Posted

It's L2Siege.java ,Change Control During Siege Chapter.

Do you still want a pastebin?

 

place it inside the bypass of the npc not inside the l2siege for announcing the registration itself

  • 0
Posted

ohh w8 i see its not like that sorry for that...

 

what you should try is to find the methods addDefender and addAttacker and inside a for loop add the announce  there

 

what i mean is:

for (L2PcInstance player : L2World.getInstance().getAllPlayers().values())

    announce....

  • 0
Posted

I have already fixed the addDefender & addAttacker announce Intrepid.

My problem is that when a Clan takes a Castle,i can't figure out a way to trigger an announcement.

 

Thanks for your replies.

 

- Rev

  • 0
Posted

I have already fixed the addDefender & addAttacker announce Intrepid.

My problem is that when a Clan takes a Castle,i can't figure out a way to trigger an announcement.

 

Thanks for your replies.

 

- Rev

 

ahh sorry again its late here and im slow :D

 

than put the same loop into the endSiege method

  • 0
Posted

I can make the following:

 

Trigger an announcement when siege ends,which Clan controls the Castle.

 

But this is not what i want.

I want to:

 

Trigger an announcement,as soon as the Clan gets the control of the Castle.

  • 0
Posted

I can make the following:

 

Trigger an announcement when siege ends,which Clan controls the Castle.

 

But this is not what i want.

I want to:

 

Trigger an announcement,as soon as the Clan gets the control of the Castle.

 

ahh so when someone take over the artifact not when the siege actually ends right?

  • 0
Posted

Exactly....And this code isn't working. It doesn't recognise the

player

part.

Any way to fix this error?

 

This is the error:

errorck.jpg

 

easy fix to put it in a loop like a said :)

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Answer this question...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.



×
×
  • Create New...