Help Problem With Custom Donator/hero System

[satanoula]

Hi, i am using L2jFrozen and i tried to use this system but it seems that i cant get it to work :/

http://www.l2jfrozen.com/forum/index.php?topic=13536.0

 

The donator item was causing troubles but i aplied the fix suggested on this topic and it solved the problem.

 

Now the problem is with hero item, let me explain...

if i use hero coin i get hero status etc just fine and i see the database update my hero end date as it should

when i relog i am no longer a hero BUT in database i can still see hero value 1 and my remaining time

also if i try to use hero coin again then donator coin in some point the data in DB get messed up, it set hero 0 and updates the donator end time and the opposit....

 

Did anyone have this problem or have any idea what can be the issue??

 

Thanks in advance!

[SweeTs]

Sounds like a missing crap @ EnterWorld or messy PcInstance, the 'load' part.

Edited October 25, 2015 by SweeTs

[satanoula]

I didnt touch enterworld at all.

Plus that if hero_end_date is 0 (witch is forever) the character logs in as a hero normally no problem.

 

maybe there is something wrong here?

 

HeroCustomItem.java

 			if(player == null)
 				return;
 
-			con = L2DatabaseFactory.getInstance().getConnection(false);
-			PreparedStatement stmt = con.prepareStatement(INSERT_DATA);
-
+			con = L2DatabaseFactory.getInstance().getConnection(false);			
+			
+			PreparedStatement stmt = con.prepareStatement(INSERT_DATA);			
+			
 			stmt.setInt(1, player.getObjectId());
 			stmt.setString(2, player.getName());
 			stmt.setInt(3, 1);
 			stmt.setInt(4, player.isNoble() ? 1 : 0);
 			stmt.setInt(5, player.isDonator() ? 1 : 0);
-			stmt.setLong(6, heroTime == 0 ? 0 : System.currentTimeMillis() + heroTime);
+			
+			if (player.isHero())
+			{
+				stmt.setLong(6, player.getHeroDate(player.getObjectId()) + heroTime);
+			}
+			else
+			{
+				stmt.setLong(6, heroTime == 0 ? 0 : System.currentTimeMillis() + heroTime);
+			}
+			
+			stmt.setLong(7, player.isDonator() ? 0 : player.getDonatorDate(player.getObjectId()));
 			stmt.execute();
 			stmt.close();
 			stmt = null;

and here is the part for L2PcInstance.java

+	
 	/**
+	 * Gets Hero end time
+	 * 
+	 * @return time, 0 if time is infinitive, or -1 if error
+	 */
+	
+	public long getHeroDate(int obj_Id)
+	{
+		Connection con = null;
+		int out = -1;
+		
+		try
+		{
+			con = L2DatabaseFactory.getInstance().getConnection(false);
+			PreparedStatement stmt = con.prepareStatement("SELECT hero_end_date FROM characters_custom_data WHERE obj_Id=" + obj_Id);
+			ResultSet rs = stmt.executeQuery();
+			if (rs.next())
+			{
+				out = rs.getInt(1);
+			}
+			rs.close();
+			rs = null;
+			stmt.close();
+			stmt = null;
+		}
+		catch (Exception e) {
+			if(Config.ENABLE_ALL_EXCEPTIONS)
+				e.printStackTrace();
+			
+			_log.log(Level.SEVERE, "Error: could not get hero_end_date from database: ", e);
+		}
+		finally {
+			CloseUtil.close(con);			
+			con = null;
+		}
+		return out;
+	}
+	
+	/**

btw thanks for ur interest in this :)

Edited October 25, 2015 by satanoula

[satanoula]

Anyone have made this script work on L2j Frozen?