Not trying to flame, but none of these profiles will run correctly, even with the fixes suggested. It's a good start though! You need to keep in mind every aspect of a player getting to that location, performing the objective, and then getting on to the next one, or turning it in.
For starters, besides quests that involve simply killing things, Honorbuddy won't understand their objective, and you have to provide behaviors to simulate those actions. Secondly, if you want the bot to pick up quests when the NPC's don't have specific dailies every day, you tell the bot to pickup whatever they have, if they have anything.
Code:
<While Condition="HasQuestAvailable([COLOR=#ff0000]58228[/COLOR])" >
<CustomBehavior File="InteractWith" MobId="[COLOR=#ff0000]58228[/COLOR]" [COLOR=#800080]X="1553.209" Y="-2564.307" Z="151.4416" />[/COLOR]
<CustomBehavior File="RunMacro" Macro="/click GossipTitleButton1" />
<CustomBehavior File="RunMacro" Macro="/click QuestFrameAcceptButton" />
</While>
The red text would be the NPC it will be trying to get quests from, the coordinates are for it's location, as the bot may not know where they are at this point, and will be confused if the player is far from them (for some reason).
Also, you need to think in a linear fashion, and keep in mind the user may jump around the entire profile. So here's a snippet of code.
Code:
<If Condition="HasQuest(31705)">
<UseItem ItemId="89113" QuestName="Angst vor der Nadel" QuestId="31705" X="1704.442" Y="-2525.799" Z="155.327" />
<Objective QuestName="Angst vor der Nadel" QuestId="31705" Type="KillMob" MobId="65616" KillCount="1" />
</If>
The problem with this, is what if the player stopped the profile, did something, and is now in the Dread Wastes or whatever... (Pretend your other tags don't exist or are perfectly nested in delicious and acceptable conditions).. The bot will not know that it needs to move anywhere, so it will just try to use that item, which will probably not work too well being a few thousand meters in the wrong direction.
Horrible explanations above aside, you need to assume the bot has no clue what common sense is, and you need to explain to it every possible scenario a player could go through. Before you can get good with thinking these things out, it's acceptable to put every step into every If condition. While it may do this-or-that one too many times, it's better than breaking your profile. Kick has a blank QO with lots of stuff you could use and learn from, however these profiles are far from functional. I'm going to try to release one of them soon as a buyer of mine bailed on me, but they take a lot of time considering these new lengthy dailies they give us.