not ready to say bye bye to USE_THE_NICK

This commit is contained in:
psyc://psyced.org/~lynX 2009-03-07 18:00:39 +01:00
parent 1b9b468ecf
commit 40adfaea01
3 changed files with 16 additions and 7 deletions

View File

@ -13,6 +13,15 @@ ________________________________________________________________________
________________________________________________________________________ ________________________________________________________________________
== currently being inspected =========================================== == currently being inspected ===========================================
¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯ ¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯
- neither USE_THE_NICK or USE_AUTOALIAS work flawlessly. in the case of
the latter, _nick either displays without uniform for remote people,
or it occasionally shows a room's nick instead of the user's nick.
they way to solve this is to get rid of _nick in the protocol, it seems.
therefore we need to upgrade all servers to properly be able to display
_source, _target and _context in psyctext before we can start migrating
templates and removing _nick vars. during this tradition we keep the
USE_THE_NICK code running. several "invite issues" should be solved once
nicks are gone.
________________________________________________________________________ ________________________________________________________________________
== psyced 1.0 ========================================================== == psyced 1.0 ==========================================================
¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯ ¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯¯

View File

@ -220,9 +220,6 @@
# define NEW_UNLINK # define NEW_UNLINK
# define NEW_RENDER # define NEW_RENDER
# define MUCSUC # define MUCSUC
#else
// the old nick code seems to introduce the remote-part bug
# define USE_THE_NICK
#endif #endif
#define GAMMA // code that has left BETA and is in production use #define GAMMA // code that has left BETA and is in production use
@ -241,6 +238,9 @@
# define HTFORWARD // let person entity buffer output for http usage # define HTFORWARD // let person entity buffer output for http usage
# endif # endif
#else #else
// the old nick code seems to introduce the remote-part bug
// but if we don't use it, psyced will render [_nick] wrong in many places
# define USE_THE_NICK
//# define PRE_SPEC // things that changed during the spec process //# define PRE_SPEC // things that changed during the spec process
//# ifndef __PIKE__ //# ifndef __PIKE__
//# define USE_LIVING //# define USE_LIVING

View File

@ -73,7 +73,7 @@ varargs string psyctext(string s, mapping m, vastring data,
break; break;
case "_source": case "_source":
// should this support _source_relay? var inheritance! // should this support _source_relay? var inheritance!
#ifdef USE_THE_NICK #ifdef USE_THE_NICK_TOO_MUCH
r += p + (nick || m["_nick"] || m["_source"] r += p + (nick || m["_nick"] || m["_source"]
|| UNIFORM(source) || "?"); || UNIFORM(source) || "?");
#else #else
@ -84,7 +84,7 @@ varargs string psyctext(string s, mapping m, vastring data,
#endif #endif
break; break;
case "_source_relay": case "_source_relay":
#ifdef USE_THE_NICK #ifdef USE_THE_NICK_TOO_MUCH
r += p + (nick || m["_nick"] || m["_source_relay"] || "?"); r += p + (nick || m["_nick"] || m["_source_relay"] || "?");
#else #else
q = m["_source_relay"] || "?"; q = m["_source_relay"] || "?";
@ -94,7 +94,7 @@ varargs string psyctext(string s, mapping m, vastring data,
#endif #endif
break; break;
case "_target": case "_target":
#ifdef USE_THE_NICK #ifdef USE_THE_NICK_TOO_MUCH
r += p + (m["_nick_target"] || m["_target"] || "?"); r += p + (m["_nick_target"] || m["_target"] || "?");
#else #else
q = m["_target"] || "?"; q = m["_target"] || "?";
@ -104,7 +104,7 @@ varargs string psyctext(string s, mapping m, vastring data,
#endif #endif
break; break;
case "_context": case "_context":
#ifdef USE_THE_NICK #ifdef USE_THE_NICK_TOO_MUCH
r += p + (m["_nick_place"] || m["_context"] || "?"); r += p + (m["_nick_place"] || m["_context"] || "?");
#else #else
q = m["_context"] || "?"; q = m["_context"] || "?";