r6914 | TaeKwon Mission Skill

This place is for Closed bug reports only. NOT for asking help!

Moderators: Moderators, Developers

midnytblu
Developers
Developers
Posts: 90
Joined: 14 Apr 2008, 09:37
Noob?: No
Location: prt_fild08 134 362

r6914 | TaeKwon Mission Skill

#1 Post by midnytblu »

using the TaeKwon Mission skill results in a garbled message where it's supposed to display the current monster mission.

Image

works well on 2.0.6.1.

EDIT:
current mission in-game is Aster.
Last edited by midnytblu on 01 Oct 2009, 23:38, edited 1 time in total.
kali
OpenKore Monk
OpenKore Monk
Posts: 457
Joined: 04 Apr 2008, 10:10

Re: SVN | TaeKwon Mission Skill

#2 Post by kali »

Thanks midnytblue, what's the svn revision number?

If you have tortoise svn you can right click on the versioned openkore folder and get it from one of the menus there.
Got your topic trashed by a mod?

Trashing topics is one click, and moving a topic to its proper forum is a lot harder. You expend the least effort in deciding where to post, mods expend the least effort by trashing.

Have a nice day.
midnytblu
Developers
Developers
Posts: 90
Joined: 14 Apr 2008, 09:37
Noob?: No
Location: prt_fild08 134 362

Re: SVN | TaeKwon Mission Skill

#3 Post by midnytblu »

hello sir kali,

i got r6914. i'll change the topic title.
kali
OpenKore Monk
OpenKore Monk
Posts: 457
Joined: 04 Apr 2008, 10:10

Re: r6914 | TaeKwon Mission Skill

#4 Post by kali »

Thanks :)

If you wanna make a stab at resolving the bug, try this:
Revert to the previous revision one rev at a time, and find out which revision has it working, and which revision immediately after is causing the bug. Odds are, the changes between those revisions caused the regression.

I know this might take some time, but doing so will help nail down the exact problem :)
Got your topic trashed by a mod?

Trashing topics is one click, and moving a topic to its proper forum is a lot harder. You expend the least effort in deciding where to post, mods expend the least effort by trashing.

Have a nice day.
midnytblu
Developers
Developers
Posts: 90
Joined: 14 Apr 2008, 09:37
Noob?: No
Location: prt_fild08 134 362

Re: r6914 | TaeKwon Mission Skill

#5 Post by midnytblu »

^this is actually the first time i used my ranker on SVN. not sure if the bug was already present beforehand but i'll give it a shot as you suggested.

/no1
Technology
Super Moderators
Super Moderators
Posts: 801
Joined: 06 May 2008, 12:47
Noob?: No

Re: r6914 | TaeKwon Mission Skill

#6 Post by Technology »

It happened in revision 6840 i believe.

It has something todo with the change from:

Code: Select all

'020E' => ['taekwon_mission_receive', 'Z24 a4 c1', [qw(monName ID value)]],

sub taekwon_mission_receive {
	 my ($self, $args) = @_;
	message T("TaeKwon Mission : ".$args->{monName}."(".$args->{value}."\%)"."\n"), "info";
} 
to the more advanced:

Code: Select all

'020E' => ['teakwon_packets', 'Z24 a4 C2', [qw(name ID value flag)]],

# TODO test if we must use ID to know if the packets are meant for us.
sub teakwon_packets {
	my ($self, $args) = @_;
	if ($args->{flag} == 0) {
		# Info about Star Gladiator save map: Map registered
		my $string = ($args->{value} == 1) ? "Sun" : ($args->{value} == 2) ? "Moon" : ($args->{value} == 3) ? "Stars" : "unknown";
		message TF("You have now marked: %s as Place of the %s.\n", $args->{name}, $string), "info";
	} elsif ($args->{flag} == 1) {
		# Info about Star Gladiator save map: Information
		my $string = ($args->{value} == 1) ? "Sun" : ($args->{value} == 2) ? "Moon" : ($args->{value} == 3) ? "Stars" : "unknown";
		message TF("%s is marked as Place of the %s.\n", $args->{name}, $string), "info";
	} elsif ($args->{flag} == 10) {
		# Info about Star Gladiator hate mob: Register mob
		my $string = ($args->{value} == 1) ? "Sun" : ($args->{value} == 2) ? "Moon" : ($args->{value} == 3) ? "Stars" : "unknown";
		message TF("You have now marked %s as Target of the %s.\n", $args->{name}, $string), "info";
	} elsif ($args->{flag} == 11) {
		# Info about Star Gladiator hate mob: Information
		my $string = ($args->{value} == 1) ? "Sun" : ($args->{value} == 2) ? "Moon" : ($args->{value} == 3) ? "Stars" : "unknown";
		message TF("%s is marked as Target of the %s.\n", $args->{name}, $string);
	} elsif ($args->{flag} == 20) {
		#Info about TaeKwon Do TK_MISSION mob
		message TF("TaeKwon Mission : %s (%)"."\n", monsterName($args->{ID}), $args->{value}), "info";
	} elsif ($args->{flag} == 30) {
		#Feel/Hate reset
		message T("Your Hate and Feel targets have been resetted.\n"), "info";
	} else {
		warning TF("flag: %s gave unknown results in: %s\n", $args->{flag}, $self->{packet_list}{$args->{switch}}->[0]);
	}
}
I'm in class now so don't have access to stuff
The length of the packet seems to be 32 so thats right. (2+24+4+2)
the diff

try changing:

Code: Select all

message TF("TaeKwon Mission : %s (%)"."\n", monsterName($args->{ID}), $args->{value}), "info";
to

Code: Select all

message TF("TaeKwon Mission: %s (%)\n", $args->{name}, $args->{value}), "info";
(the cause might be the concatination but probably its monsterName($args->{ID}) )

eA's code however shows (note the mob_id):

Code: Select all

void clif_mission_info(struct map_session_data *sd, int mob_id, unsigned char progress)
{
	int fd=sd->fd;
	WFIFOHEAD(fd,packet_len(0x20e));
	WFIFOW(fd,0)=0x20e;
	strncpy((char*)WFIFOP(fd,2),mob_db(mob_id)->jname, NAME_LENGTH);
	WFIFOL(fd,26)=mob_id;
	WFIFOB(fd,30)=progress; //Message to display
	WFIFOB(fd,31)=20;
	WFIFOSET(fd, packet_len(0x20e));
}
One ST0 to rule them all? One PE viewer to find them!
One ST_kRO to bring them all and in the darkness bind them...

Mount Doom awaits us, fellowship of OpenKore!
User avatar
kLabMouse
Administrator
Administrator
Posts: 1301
Joined: 24 Apr 2008, 12:02

Re: r6914 | TaeKwon Mission Skill

#7 Post by kLabMouse »

Anybody noticed that Kore say "Unknown NPC" ?
midnytblu
Developers
Developers
Posts: 90
Joined: 14 Apr 2008, 09:37
Noob?: No
Location: prt_fild08 134 362

Re: r6914 | TaeKwon Mission Skill

#8 Post by midnytblu »

^i just logged on when i took the screenshot. don't mind that :D
midnytblu
Developers
Developers
Posts: 90
Joined: 14 Apr 2008, 09:37
Noob?: No
Location: prt_fild08 134 362

Re: r6914 | TaeKwon Mission Skill

#9 Post by midnytblu »

@Technology
sorry but where can i find this?
Technology wrote: try changing:

Code: Select all

message TF("TaeKwon Mission : %s (%)"."\n", monsterName($args->{ID}), $args->{value}), "info";
to

Code: Select all

message TF("TaeKwon Mission: %s (%)\n", $args->{name}, $args->{value}), "info";
(the cause might be the concatination but probably its monsterName($args->{ID}) )
i checked Receive.pm but this line is not present.
thanks.
Technology
Super Moderators
Super Moderators
Posts: 801
Joined: 06 May 2008, 12:47
Noob?: No

Re: r6914 | TaeKwon Mission Skill

#10 Post by Technology »

src\Network\Receive\ServerType0.pm
One ST0 to rule them all? One PE viewer to find them!
One ST_kRO to bring them all and in the darkness bind them...

Mount Doom awaits us, fellowship of OpenKore!