[2.0] House Paints - Paint just about anything...

[oiii88]

Not sure why but when I use my player I am still able to paint buildings in luna using sandstone paint . And this is after updating with your latest post.

[Lokai]

Originally Posted by Hotshot

can you add here what you changed and where.. so i can copy and paste

I added this near the beginning of the OnTarget method:

PHP Code:

 
BaseHouse house = Multis.BaseHouse.FindHouseAt(target);
if (house == null || from != house.Owner || from.AccessLevel < AccessLevel.GameMaster)
{
from.SendMessage("That is not part of your house!");
return;
} 


Originally Posted by purplemouse91

UHOH tested this and came back to it an hour later and the walls turned back to its original color...

There must be a decay value on the PaintedItem I didn't notice....

Originally Posted by oiii88

Not sure why but when I use my player I am still able to paint buildings in luna using sandstone paint . And this is after updating with your latest post.

That does not make any sense, unless someone can see what I missed in that code above...

[oiii88]

Its odd you mentioned your addition because I looked at what I had downloaded and it did not have that line in it . I will edit mine accordingly. Guess I deleted the wrong file.

[oiii88]

OK I replaced the script with the one you have posted now and I got this error.

Errors:
+ Customs/House Paint System/PaintCan.cs:
CS0103: Line 140: The name 'target' does not exist in the current context

[Rosey1]

This looks great Lokai! I can't wait to try it!

[kirax2]

Wow...I didn't think this could be done. I'm really impressed! When I eventually get my shard up and running, I think this will be an awesome addition. Thank you!

-kirax2

[Rosey1]

I got that same target compile error btw : (

[Lokai]

Originally Posted by oiii88

OK I replaced the script with the one you have posted now and I got this error.

Errors:
+ Customs/House Paint System/PaintCan.cs:
CS0103: Line 140: The name 'target' does not exist in the current context

Sorry, I must have made a last second copy/paste before posting. That should read 'targeted', not 'target'. Change the line 140 to say:

BaseHouse house = Multis.BaseHouse.FindHouseAt(targeted);

[Lokai]

Originally Posted by kirax2

Wow...I didn't think this could be done. I'm really impressed! When I eventually get my shard up and running, I think this will be an awesome addition. Thank you!

-kirax2

Folks, here is the person that started this whole thing...

So, blame her if your shard is all whacked out pretty! hehe.

[dagan1545]

Nice script!!
But i did have a problem with line 30 in the paintcan.cs /theres no return.
Other then that everything looks great.

[Hotshot]

K, on the Target issue... How bout adding a contract deed that will let people who obtain these deeds, sell there services ? Kool Idea i think... maybe create a creature for these, almost like Fletcher tools. Maybe a Furious House painter, make him red and randomly drop House Tubs/Paint cans. also... is the brick problem solved, Other paint tub dont dye/paint it. maybe script a tub that has.. brick
Nice script again.... ++karma

[Hotshot]

Errors:
+ Custom Scripts/Painting Houses/House Paint System/PaintCan.cs:
CS1502: Line 140: The best overloaded method match for 'Server.Multis.BaseHo
use.FindHouseAt(Server.Mobile)' has some invalid arguments
CS1503: Line 140: Argument '1': cannot convert from 'object' to 'Server.Mobi
le'
Scripts: One or more scripts failed to compile or no script files were found.
- Press return to exit, or R to try again.

[Lokai]

Originally Posted by Hotshot

K, on the Target issue... How bout adding a contract deed that will let people who obtain these deeds, sell there services ? Kool Idea i think... maybe create a creature for these, almost like Fletcher tools. Maybe a Furious House painter, make him red and randomly drop House Tubs/Paint cans. also... is the brick problem solved, Other paint tub dont dye/paint it. maybe script a tub that has.. brick
Nice script again.... ++karma

"Other House Paint" is for Paper, Bamboo, Rattan, and Hides.

Brick should be covered under Stone, not sure. What are the ItemID's of some of the Brick items?

[Lord_Greywolf]

nice script and great idea

only thing is for people that have a decent player base (20+ regular players) and you allow more than 1 house per player
the number of items can add up a lot
and if a persom just paints the out side edge (or interisor of outside) of a custom 18x18 3 story that is alone is 144 items, could total closr to 1000 on a well build home (and if this does work on a castle - whoa - big numbers)
this will add to the lag of the house, and of the area too
ans of course save times & shard item count
besides having to wipe the stuff once the house is gone

so be carefull on it, if your players start to complain of lag, etc

[Erikdalam]

nice job,Thank

[Lokai]

Sorry about this guys. Everything was working, until I tried to fix the problem with players being able to paint all over the world instead of their houses.

I have withdrawn the download until I can fix several issues with crashes and odd behavior.

[Lokai]

ORIGINAL POST UPDATED AND NEW DOWNLOAD POSTED:

5/14/2007:

- Fixed many errors including null exceptions.
- Made it so Painted walls are movable (but you still can't move them)
- Painted walls are locked down when created.
- Users must Release the old paint in order to repaint it.
- Hopefully this will decrease Item-lag.
- Fixed targeting issues.
- Players may now paint from just outside their house, but must still target a house they own.
- GMs can still paint anywhere.
- Fixed bug allowing players to paint all over town.

KNOWN ISSUES:

- When users log in, their Client may load the Locked-down painted items before the regular house walls. This causes the paint to disappear. Not sure if the Lockdown feature will fix this. If anyone has an idea on setting a priority level or something so that the Painted walls display last, let me know.
- Painted Walls look movable. This is NOT a bug. Think of it like this: if you don't SEAL the paint job, it will wear off (ie, Decay.) Users must Secure them to prevent them from fading. (An added bonus is that this may reduce excessive painting, since it lowers the total Lockdowns available.)

[Lokai]

FIX for Painted Walls staying after House is destroyed:


In BaseHouse.cs, add Lokai's House Paints Region as below:

PHP Code:

   if ( m_LockDowns != null )
   {
    for ( int i = 0; i < m_LockDowns.Count; ++i )
    {
     Item item = (Item)m_LockDowns[i];
     if ( item != null )
     {
      item.IsLockedDown = false;
      item.IsSecure = false;
      item.Movable = true;
      item.SetLastMoved();
      #region Lokai's House Paints
      if (item is PaintedItem)
       item.Delete();
      #endregion
     } 


[Macil]

Originally Posted by Lokai

FIX for Painted Walls staying after House is destroyed:


In BaseHouse.cs, add Lokai's House Paints Region as below:

PHP Code:

   if ( m_LockDowns != null )
   {
    for ( int i = 0; i < m_LockDowns.Count; ++i )
    {
     Item item = (Item)m_LockDowns[i];
     if ( item != null )
     {
      item.IsLockedDown = false;
      item.IsSecure = false;
      item.Movable = true;
      item.SetLastMoved();
      #region Lokai's House Paints
      if (item is PaintedItem)
       item.Delete();
      #endregion
     } 


Uh? Its still leaving the walls and stuff. =(

[Lokai]

Originally Posted by Macil

Uh? Its still leaving the walls and stuff. =(

GMs will leave walls, players will not.

[Macil]

Originally Posted by Lokai

GMs will leave walls, players will not.

PAH! Details!

Everything's all good here now. Karma time.

[purplemouse91]

Originally Posted by Lokai

FIX for Painted Walls staying after House is destroyed:


In BaseHouse.cs, add Lokai's House Paints Region as below:

PHP Code:

   if ( m_LockDowns != null )
   {
    for ( int i = 0; i < m_LockDowns.Count; ++i )
    {
     Item item = (Item)m_LockDowns[i];
     if ( item != null )
     {
      item.IsLockedDown = false;
      item.IsSecure = false;
      item.Movable = true;
      item.SetLastMoved();
      #region Lokai's House Paints
      if (item is PaintedItem)
       item.Delete();
      #endregion
     } 


Were exactly do we place this...

[paggylee]

Hi,

I added the lines you said to to my basehouse.cs script.. got and error.. I am not exactly sure where I need to put this in on the cs script.. I can send you my basehouse.cs script if you desire, and the error I am getting on it is..

Error:

RunUO - [www.runuo.com] Version 2.0, Build 2357.32527
Core: Running on .NET Framework Version 2.0.50727
Scripts: Compiling C# scripts...failed (1 errors, 0 warnings)
Errors:
+ Added Customs/Loose Scripts/BaseHouse.cs:
CS1014: Line 2776: A get or set accessor expected
CS1513: Line 2774: } expected
Scripts: One or more scripts failed to compile or no script files were found.
- Press return to exit, or R to try again.


Thank you for you help
Kyss/Peg

[Lokai]

Originally Posted by purplemouse91

Were exactly do we place this...

Below this:

PHP Code:

item.SetLastMoved(); 


place this:

PHP Code:

      #region Lokai's House Paints 
      if (item is PaintedItem) 
       item.Delete(); 
      #endregion 


[Lokai]

Originally Posted by paggylee

Hi,

I added the lines you said to to my basehouse.cs script.. got and error.. I am not exactly sure where I need to put this in on the cs script.. I can send you my basehouse.cs script if you desire, and the error I am getting on it is..

Error:

RunUO - [www.runuo.com] Version 2.0, Build 2357.32527
Core: Running on .NET Framework Version 2.0.50727
Scripts: Compiling C# scripts...failed (1 errors, 0 warnings)
Errors:
+ Added Customs/Loose Scripts/BaseHouse.cs:
CS1014: Line 2776: A get or set accessor expected
CS1513: Line 2774: } expected
Scripts: One or more scripts failed to compile or no script files were found.
- Press return to exit, or R to try again.


Thank you for you help
Kyss/Peg

you only add this part:

PHP Code:

      #region Lokai's House Paints 
      if (item is PaintedItem) 
       item.Delete(); 
      #endregion