Jump to content

My rAthena suggestions


Recommended Posts


  • Group:  Members
  • Topic Count:  19
  • Topics Per Day:  0.00
  • Content Count:  713
  • Reputation:   70
  • Joined:  11/08/11
  • Last Seen:  

Some things that I want to see in rA:

- add a #ifdef GEOIP acording to this topic

- fix battleground buttom

- fix party booking button

- fix items (14597,14598,14602,14603,14604,14605 and other scrolls maybe) so they display their respective buff icon like on official servers

Also I would like to see Reins of mount implemented as in official servers, not using a script command but using the sc_start thing

already done \/

- add this "You can no longer set up purchase shops / vends within a 7x7 area around any NPC's." from 8/17/2011 kRO Maintenance

- add the item stack system from eA into rA

- add Falcon Flute item

- add extended super novice

- add support to new carts

- add sorcerer and genetic skills and summons

- add support to baby 3rd classes

- this patch from eA RE branch

- add full support to at least 2011-11-22aRagexeRE

- add MvP tomb

- add homunculus S & skills

- split npc/mob & warps into pre-re and re

- remove leftover code from r16279 from pc.c

- rework the pre-re npc folder so it is really pre-re

- remove leftover REMODE comments over conf files since it is called only RENEWAL now

- add ALL_PARTYFLEE from eA RE branch

- add support to favorite item tab

- fix body relocation animation

- make rAthena default in all files, since there are places where it is Rathena

- add kagerou & oboro skills

- add a RENEWAL #ifdef to change @go 5 coord (128, 114 if pre-re) and (128, 146 if re)

- add monster hp bar from 4/04/2012 - Maintenance

- fix tetra vortex animation

- add some useful src documentations from lighta

- add some hard-coded text from src into msg_athena so we may translate them easier, ex: the msg from item cooldown etc

- add seperate map_cache.dat for RE and PRE-RE. because of Old Izlude and Old Alberta

Edited by EvilPuncker
Link to comment
Share on other sites

  • 2 weeks later...

  • Group:  Members
  • Topic Count:  69
  • Topics Per Day:  0.02
  • Content Count:  1315
  • Reputation:   372
  • Joined:  12/10/11
  • Last Seen:  

- make rAthena default in all files, since there are places where it is Rathena (Still missing in the ASCII art)

It has been decided a long time ago that ASCII art will no longer be used in any of the files and will only be displayed in the _servers consoles.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  19
  • Topics Per Day:  0.00
  • Content Count:  713
  • Reputation:   70
  • Joined:  11/08/11
  • Last Seen:  

- make rAthena default in all files, since there are places where it is Rathena (Still missing in the ASCII art)

It has been decided a long time ago that ASCII art will no longer be used in any of the files and will only be displayed in the _servers consoles.

that is really what I mean!! the consoles

Link to comment
Share on other sites

  • 3 weeks later...

  • Group:  Members
  • Topic Count:  19
  • Topics Per Day:  0.00
  • Content Count:  713
  • Reputation:   70
  • Joined:  11/08/11
  • Last Seen:  

bump for the console ascii art and add the item stack system from eA into rA

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  40
  • Topics Per Day:  0.01
  • Content Count:  587
  • Reputation:   104
  • Joined:  11/19/11
  • Last Seen:  

+1

yeah, if Implemented item stack limit system would be more convenient for setting

14635

  • Upvote 2
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  75
  • Topics Per Day:  0.02
  • Content Count:  2223
  • Reputation:   593
  • Joined:  10/26/11
  • Last Seen:  

eathenabot will merge eAthena's recent changes when I have time.

If you (or anyone reading this) would like to help, you can apply for a developer position.

Additional requirements:

  • good understanding of all rAthena files (source code, scripts, databases, everything)
  • very familiar with Subversion
  • detail-oriented

Edit: oh, I didn't realize r14635/branches/renewal/ was skipped.

I'll add it to the todo list. Thanks for letting us know!

Edited by Brian
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  19
  • Topics Per Day:  0.00
  • Content Count:  713
  • Reputation:   70
  • Joined:  11/08/11
  • Last Seen:  

That is not what we are talking about Brian xD it is a feature from eA RE branch :P

Link to comment
Share on other sites

  • 2 weeks later...

  • Group:  Members
  • Topic Count:  75
  • Topics Per Day:  0.02
  • Content Count:  2223
  • Reputation:   593
  • Joined:  10/26/11
  • Last Seen:  

r16279 /no1

* Merged /branches/renewal/ r14635 to /trunk (follow up to r15060) pid:106973
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  40
  • Topics Per Day:  0.01
  • Content Count:  587
  • Reputation:   104
  • Joined:  11/19/11
  • Last Seen:  

r16279 /no1

* Merged /branches/renewal/ r14635 to /trunk (follow up to r15060) pid:106973

Hi~~Brian,

First ,thx for Implemented that system.

I think the rune stone amount no longer need to define

we can use this system to limit rune stone amount :)

src/map/pc.c

   if( itemdb_is_rune(item_data->nameid) ) {
       int rune = pc_search_inventory(sd,item_data->nameid);
       if( ( rune >= 0 && sd->status.inventory[rune].amount + amount > MAX_RUNE ) ||
           ( rune == -1 && amount > MAX_RUNE )
               ) {
           clif_msgtable(sd->fd,0x61b);
           return 1;
       }
   }

a/src/map/skill.c

   if( skill_id == RK_RUNEMASTERY ) {
       int temp_qty, skill_lv = pc_checkskill(sd,skill_id);
       if( skill_lv == 10 ) temp_qty = 1 + rnd()%3;
       else if( skill_lv > 5 ) temp_qty = 1 + rnd()%2;
       else temp_qty = 1;
       for( i = 0; i < MAX_INVENTORY; i++ ) {
           if( sd->status.inventory[i].nameid == nameid ) {
               if( sd->status.inventory[i].amount >= MAX_RUNE ) {
                   clif_msgtable(sd->fd,0x61b);
                   return 0;
               } else {
                   /**
                    * the amount fits, say we got temp_qty 4 and 19 runes, we trim temp_qty to 1.
                    **/
                   if( temp_qty + sd->status.inventory[i].amount >= MAX_RUNE )
                       temp_qty = MAX_RUNE - sd->status.inventory[i].amount;
               }
               break;
           }
       }
       qty = temp_qty;
   }

src/map/config/classes/swordsman.h

/// rune knight
///
/// maximum number of runes that a rune knight character can carry at any given time
/// default: 20
#define MAX_RUNE 20

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  19
  • Topics Per Day:  0.00
  • Content Count:  713
  • Reputation:   70
  • Joined:  11/08/11
  • Last Seen:  

better to remove that one and let only the item_stack stay =D

Link to comment
Share on other sites

  • 2 weeks later...

  • Group:  Members
  • Topic Count:  4
  • Topics Per Day:  0.00
  • Content Count:  72
  • Reputation:   4
  • Joined:  01/22/12
  • Last Seen:  

i hope all your suggestions will be implemented. :) and reconsider more suggestions for rathena's development. /ok

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  19
  • Topics Per Day:  0.00
  • Content Count:  713
  • Reputation:   70
  • Joined:  11/08/11
  • Last Seen:  

atm what I want is support to favorite item tab xD and more suggestions would be added as soon as I think of :P but other ppl are welcome to give suggestions aswell

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  26
  • Topics Per Day:  0.01
  • Content Count:  2244
  • Reputation:   182
  • Joined:  11/19/11
  • Last Seen:  

favorite tab should be in soon once ind fully recovers

Link to comment
Share on other sites

  • 2 weeks later...

  • Group:  Members
  • Topic Count:  13
  • Topics Per Day:  0.00
  • Content Count:  99
  • Reputation:   10
  • Joined:  11/21/11
  • Last Seen:  

Some things that I want to see in rA:

Renewal Defense

HP/SP rebalance

Body Relocation corrected

Tetra Vortex show black image

Renewal ASPD

battleground and party booking button working

Post-balance skill formulas

Edited by michieru
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  94
  • Topics Per Day:  0.02
  • Content Count:  2192
  • Reputation:   252
  • Joined:  11/11/11
  • Last Seen:  

@michi

1) Fix body relocation yourself found in the source.

2) Textra Vortex shows that because it's the new clients. You either get Judas' package that he has that fixed that problem or you fix it yourself.

3) Renewal ASPD is already being worked on, look at the topic made by mal.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  26
  • Topics Per Day:  0.01
  • Content Count:  2244
  • Reputation:   182
  • Joined:  11/19/11
  • Last Seen:  

1) I think he's talking about the animation. We need the packets, and then update the src code somehow

2) Rytech talked about the code for tetra vortex is coded wrong, so possible src implementation could work

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  13
  • Topics Per Day:  0.00
  • Content Count:  99
  • Reputation:   10
  • Joined:  11/21/11
  • Last Seen:  

thats exactly what I mean thx Judas =)

and for the last one I just put what I want to see in rA official svn. I already know that manuflet working on. The only things I can do it's provide iRO information, I have many account on it know and I can test many things. But I can't test aspd on iro cause it's not the same as kRO

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  94
  • Topics Per Day:  0.02
  • Content Count:  2192
  • Reputation:   252
  • Joined:  11/11/11
  • Last Seen:  

Ahh, alright I see.

And for Body relo problem, try: http://mysterious-project.googlecode.com/svn/trunk/Snap-BodyRelo.txt

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  13
  • Topics Per Day:  0.00
  • Content Count:  99
  • Reputation:   10
  • Joined:  11/21/11
  • Last Seen:  

I already have the patch =) thx

Link to comment
Share on other sites

  • 2 weeks later...

  • Group:  Members
  • Topic Count:  19
  • Topics Per Day:  0.00
  • Content Count:  713
  • Reputation:   70
  • Joined:  11/08/11
  • Last Seen:  

added some (thanks to michieru), updated some and bumping with this one

I would like to see Reins of mount implemented as in official servers, not using a script command but using the sc_start thing

Link to comment
Share on other sites


  • Group:  Forum Moderator
  • Topic Count:  93
  • Topics Per Day:  0.02
  • Content Count:  10013
  • Reputation:   2345
  • Joined:  10/28/11
  • Last Seen:  

erm...since this is a suggestion thread for rAthena..xD

i would like to suggest to improve a script command..

getmonsterinfo()
strmobinfo()

if possible..make this 2 script command wont spam ( when loop is used ) error in map server if the monster id is not exist in the database.

even though now we can detect it when it return null or nothing after it is used on non-exit monster

but the map server will show error and stopped the npc to continue work further ( if keep loop for non-exist monster )..

and since we got rentitem() why not add in this in future ? xD

rentitem2()

work exactly like rentitem() but have extra parameter like getitem2()

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  155
  • Topics Per Day:  0.03
  • Content Count:  647
  • Reputation:   16
  • Joined:  11/21/11
  • Last Seen:  

i suggest seperate map_cache.dat for RE and PRE-RE. because of Old Izlude and Old Alberta..

  • Upvote 2
Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  19
  • Topics Per Day:  0.00
  • Content Count:  713
  • Reputation:   70
  • Joined:  11/08/11
  • Last Seen:  

i suggest seperate map_cache.dat for RE and PRE-RE. because of Old Izlude and Old Alberta..

added

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  2
  • Topics Per Day:  0.00
  • Content Count:  117
  • Reputation:   18
  • Joined:  01/06/12
  • Last Seen:  

i suggest seperate map_cache.dat for RE and PRE-RE. because of Old Izlude and Old Alberta..

Well I'll repeat what I wrote in bug tracker in reply to EvilPuncker's "bug/suggestion":

I don't think that's necessary since your server will be set either for PRE-RE or RE while clients have either files for old Alberta and/or old Izlude or new Alberta and/or new Izlude.

Further more, I don't think there's servers out there that keep switching between RE and PRE-RE except during testing phases because switching map cache would mean you will have to update client side files to keep files matching with server file caches anyways.

Link to comment
Share on other sites


  • Group:  Members
  • Topic Count:  19
  • Topics Per Day:  0.00
  • Content Count:  713
  • Reputation:   70
  • Joined:  11/08/11
  • Last Seen:  

that is a bug, not a suggestion, if you think that way, sorry but you need to think outside the box. lets keep on topic please.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   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...