В личных сообщениях на 15-ом ответе (просто свободное общение) кол-во Re: Re: Re: Re: Re: Re: Re: Re: стремительно растет.
Может сделать:
Re1:
Re2:
Re3:
?

СУВ.

Или Re(1) ?

Было бы очень удобно

отличная идея!

и где же и что же ?

СУВ.

Идём в forum/sources/Messenger.php, смотрим на строки(#1365)

Код

$old_title   = "Re:".$old_msg['title']; $old_title   = preg_replace( "/^(?:Re\:){1,}/i", "Re:", $old_title );

Во второй строке кривой рег, который должен удалять повторяющиеся "Re:"
Если нам это действительно нужно, то переписываем его:
$old_title = preg_replace( "/^(?:Re\:\s*)+/i", "Re:", $old_title );

Если хотим нумерацию, то заменяем обе строчки на:

Код

$old_title=preg_replace_callback("/^(Re)(?:\\(([0-9]+)\\))?/i", create_function('$test', 'return isset($test[2])? $test[1]."(".($test[2]+1).")": $test[1]."(2)";'), $old_msg['title'], 1); $old_title=($old_title==$old_msg['title'])? $old_title="Re: ".$old_msg['title']: $old_title;

title -> Re: title
Re: title -> Re(2): title
Re(n): titile -> Re(n+1): title

Нашёл свободные пять минут

А почему это не добавлено?

сделал.


проверьте! )

Хм.. Игнат и я попробовали... Странно, что теперь и Re несколько раз как раньше и цифра. Например:

Re: Re: Re: Re(2): Test

У меня не работает... (((

Я сейчас переписывался в последнем сообщении было такое:
Re: Re(2): Re: Re: Re: Re: ...

Странно... я проверил, должно работать... Посмотрите реги
Может какие пробелы стоят перед Re, Wowa попробуй так:

Код

$old_title=preg_replace_callback("/^\s*(Re)(?:\\(([0-9]+)\\))?/i", create_function('$test', 'return isset($test[2])? $test[1]."(".($test[2]+1).")": $test[1]."(2)";'), $old_msg['title'], 1); $old_title=($old_title==$old_msg['title'])? $old_title="Re: ".$old_msg['title']: $old_title;

Цитата(Sardar @ 5.5.2005, 15:42)

Странно... я проверил, должно работать... Посмотрите реги  Может какие пробелы стоят перед Re, Wowa попробуй так:

заменил
Добавлено @ 15:13
кажется, что все равно не сокращает. Проверьте, кто-нибудь, плиз

У меня не работает...

Я прикрепил файл сюда. Может так будет легче кому-нибудь подправить ошибку.
Строки:
1367
1368

Присоединённый файл ( Кол-во скачиваний: 9 )
 Messenger.php

Мистика...

Не представляю где там пробелы еще могут стоять.. можеть где \0(null) символ всетречается перед или между Re, он не берётся паттерном \s.
Если так не заработает то торжественно пойду заного учить реги, снесу апача и всё что есть, поверю что меня глючить и то что у меня работает в любых ситуациях с любой лажей в строке - это мне только кажеться...

Код

$old_title=preg_replace_callback("/^\s*([Rr]\s*[Ee])\s*(?:\s*\\(([0-9]+)\\))?/i", create_function('$test', 'return isset($test[2])? $test[1]."(".($test[2]+1).")": $test[1]."(2)";'), $old_msg['title'], 1); $old_title=($old_title==$old_msg['title'])? $old_title="Re: ".$old_msg['title']: $old_title;

Sardar, не забудь бубен заговорить

Цитата(Sardar @ 7.5.2005, 16:11)

Не представляю где там пробелы еще могут стоять

а ты уверен, что в них дело?

Ну так не бывает! Письмо од Girder

Цитата

Вот для таких вещей... не работает Re: Re(6): Re: Re(3): Re: Re: Girder

Проверяем код:

Код

$old_msg['title']="Re: Re(6): Re: Re(3): Re: Re: Girder"; $old_title=preg_replace_callback("/^\s*([Rr]\s*[Ee])\s*(?:\s*\\(([0-9]+)\\))?/i", create_function('$test', 'return isset($test[2])? $test[1]."(".($test[2]+1).")": $test[1]."(2)";'), $old_msg['title'], 1); echo $old_title=($old_title==$old_msg['title'])? $old_title="Re: ".$old_msg['title']: $old_title;

У меня результат: Re(2): Re(6): Re: Re(3): Re: Re: Girder

Загоняю его по новой, результат: Re(3): Re(6): Re: Re(3): Re: Re: Girder

Обьясните, неужели это матрица, я обкурился или чего? Ну почему у меня работает?
Добавлено @ 18:38

Цитата(Wowa @ 7.5.2005, 17:19)

а ты уверен, что в них дело?

У меня такое ощущение что у нас по разному ведут себя реги...
Добавлено @ 18:41
Если не хотим нумерации, то можно так:

Код

$old_title   = "Re:".$old_msg['title']; $old_title = preg_replace( "/^\s*(?:Re\s*\:\s*)+/i", "Re: ", $old_title );

Вырежет все Re: оставив один.

Girder, соколиный глаз засёк багу злобную:

Цитата

А вот с таким входным параметром... запусти: $old_msg['title']='ReGir' Енто косяк...

Точно косяк. Подняв в N'ый раз свой бубен нашаманил:

Код

$old_title=preg_replace_callback("/^\s*(re)\s*(?:\s*\\(([0-9]+)\\))?\s*:/i", create_function('$test', 'return isset($test[2])? $test[1]."(".($test[2]+1)."):": $test[1]."(2):";'), $old_msg['title'], 1); $old_title=($old_title==$old_msg['title'])? $old_title="Re: ".$old_msg['title']: $old_title;

Запускаем тест(в коментариях результат у меня):

Код

//$old_msg['title']="simple title"; //Re: simple title //$old_msg['title']="Re: simple title"; //Re(2): simple title //$old_msg['title']=" \t RE  (5) :  title"; //RE(6): title //$old_msg['title']=" \n rE   : (3) title"; //rE(2): (3) title //$old_msg['title']="ReGir"; //Re: ReGir //$old_msg['title']="Re:ReGir"; //Re(2):ReGir $old_title=preg_replace_callback("/^\s*(re)\s*(?:\s*\\(([0-9]+)\\))?\s*:/i", create_function('$test', 'return isset($test[2])? $test[1]."(".($test[2]+1)."):": $test[1]."(2):";'), $old_msg['title'], 1); echo $old_title=($old_title==$old_msg['title'])? $old_title="Re: ".$old_msg['title']: $old_title;

Большой пасиб 2Girder

И так проблема вроде как порешалась(Wowa слово за тобой). Но в воздухе витала мысль, а что если убрать множественные Re: из заголовков, написанных ранее, до использования фичи?

Партия не стоит на месте, обмозговав ситуацию с Girder'ом решили написать. Girder источник идеи начал раньше, потому нашли баги, обошёл грабли, вот мой вариант:

Код

preg_match("/^\s*(re)\s*(?:\(\s*([0-9]+)\s*\))?\s*:/i",$old_msg['title'],$pmre); $str=preg_replace("/^(\s*(re)\s*(?:\(\s*([0-9]+)\s*\))?\s*:)+/i", "", $old_msg['title']); $old_title=isset($pmre[2])? "Re(".($pmre[2]+1)."): {$str}": (isset($pmre[1])? "Re(2): {$str}": "Re: {$str}");

Тесты(в коментарии результаты):

Код

//$old_msg['title']="simple title"; //Re: simple title //$old_msg['title']="Re: simple title"; //Re(2): simple title //$old_msg['title']=" \t RE  (5) :  title"; //Re(6): title //$old_msg['title']=" \n rE   : (3) title"; //Re(2): (3) title //$old_msg['title']="ReGir"; //Re: ReGir //$old_msg['title']="Re:ReGir"; //Re(2):ReGir //$old_msg['title']="Re(2): Re(2): Re(2): Girder"; //Re(3): Girder //$old_msg['title']=" test Re(2): Re(2): Re(2): Girder"; //Re: test Re(2): Re(2): Re(2): Girder //$old_msg['title']="Re(2):  Re(2): test: Re(2): Girder"; //Re(3): test: Re(2): Girder preg_match("/^\s*(re)\s*(?:\(\s*([0-9]+)\s*\))?\s*:/i",$old_msg['title'],$pmre); $str=preg_replace("/^(\s*(re)\s*(?:\(\s*([0-9]+)\s*\))?\s*:)+/i", "", $old_msg['title']); echo $old_title=isset($pmre[2])? "Re(".($pmre[2]+1)."): {$str}": (isset($pmre[1])? "Re(2): {$str}": "Re: {$str}");

Оригинальный от Girder'а отличается, запости его

Енто ты... про енто?

Код

preg_match("/^\s*(re)\s*.\s*([0-9]+)\s*.\s*:/i",$old_msg['title'],$pmre);     $spre=preg_split("/((?<=[\s:])r[eå]\s*(:|\(\s*[?>0-9]+\s*\)\s*:))/i"," ".$old_msg['title']); //$spre=split("(^ *[Rr][Ee] *[:]| *[Rr][Ee] *[:]| *[Rr][Ee] *. *[0-9]+ *. *[:]|^ *[Rr][Ee] *. *[0-9]+ *. *[:])",$old_msg['title']);     $old_title=isset($pmre["2"])? "RE(".($pmre[2]+1)."): ".trim(implode("",$spre)): "RE(".count($spre)."): ".trim(implode("",$spre));

PS:

Это сообщение отредактировал(а) Girder - 19.5.2005, 23:43

Угу, только две баги, что я на PM послал надо было пофиксить

Так, какой код мне ставить?

Код Girder'a имеетм пару "багов", которые под иным углом зрения можно видеть как фичи

Первая "бага", текст "Ret2z: способ модуляции цифрового сигнала(RZ code)", перегонит в:
"Re(3): способ модуляции цифрового сигнала(RZ code)";

Цитата

>Не плохо, ты скобки () специально не обязательными сделал? Отсюда первый косяк: Угу... уж больно захотелось убивать вот такие вещи(использую один символ) Re{2} или Re[2] - ну ты понял... А вероятноять появления в заголовке "Ret2z" - посчитал слишком молой... smile PS: Ну лодырь я! smile

Проблема в том, что принимаються не только скобки (), но и любой другой символ, т.е... Re[1], Reo1i, Re(1) - это всё одно и тоже. По моему это всё же больше бага чем фича

Вторая "бага" в удалении всех Re, даже посередине заголовка:

Цитата

>$old_msg['title']="Тестируем Re(1): Re(2): Re(2): Re(2): ReGirder"; >На выходе: "Re(4): Тестируем ReGirder"; >Все промежуточные Re() являлись частью заголовка, удалять нельзя Жаль... но тоже специально заложил smile Я частенько добавлял в начало заголовка новое пояснение... т.е. было Re: Re: Girder, а при ответе добавил в начало... типо Ok Re: Re: Girder

Это уже похоже на фичу, но мне лично как то не удобно, если куча Re: будет удаляться из середины сообщения, не логично, хотя вряд ли кто часто будет набирать такое:
"Привет, траблы с " Re: " пофиксили!", удалит Re: из середины.


Мой код этих двух "проблем" не имеет, я (естественно ) за свой код

Цитата(Sardar @ 9.5.2005, 02:03)

Проблема в том, что принимаються не только скобки (), но и любой другой символ, т.е... Re[1], Reo1i, Re(1) - это всё одно и тоже. По моему это всё же больше бага чем фича

Ну хорошо... пусть будет бага PS: Хотя я считаю что бага вот на таких вещах: Re 12 : => Re(3):

Цитата(Sardar @ 9.5.2005, 02:03)

Мой код этих двух "проблем" не имеет, я (естественно  ) за свой код

А мне пофиг...

PS: Я за хоть любой вариант... лишбы поставили. Кстати... разница между кодами есть и более существенная: Т.е например:
1) Строка " re: Re(3): Re: Re: Re: Re: Re: GirderRe:"
- твой: Re(2): GirderRe:
- мой: RE(8): GirderRe: - (там правда... см. в коде выше пришлось изменить вторую строчку, старую закомментировал... - а то не совсем понравилось... как код ведет себя на некоторый строчках )
2) Строка " re ( 23 ) : Re(3): Re: Re: Re: Re: Re: GirderRe:"
- твой: RE(24): GirderRe:
- мой: RE(24): GirderRe:
3)Строка " re: Re: Re: Re: Re: Re: Re: GirderRe:"
- твой: Re(2): GirderRe:
- мой: RE(8): GirderRe:

Т.е. пропустил еще одну "фичу" Т.е. так в моем коде... если первый RE не записан в виде Re(22) - то "берется" количество Re иначе берется 22+1

PS2: Голосую за вариант Sardar-а

Это сообщение отредактировал(а) Girder - 9.5.2005, 13:07

1. Правим Messenger.php:
- a) Из кода Sardar-а убираем лишние пробельчики... то есть лучше вот так:

Код

preg_match("/^\s*(re)\s*(?:\(\s*([0-9]+)\s*\))?\s*:/i",$old_msg['title'],$pmre); $str=preg_replace("/^(\s*(re)\s*(?:\(\s*([0-9]+)\s*\))?\s*:)+/i", "", $old_msg['title']); $old_title=isset($pmre[2])? "Re(".($pmre[2]+1)."):{$str}": (isset($pmre[1])? "Re(2): {$str}": "Re: {$str}");

- б) Находим следующий код(функция view_msg):

Код

$this->output .= $this->html->Render_msg( array(                                                                   'msg'    => $msg,                                                                   'member' => $member,                                                                   'jump'   => $this->jump_html                                                            )      );

заменяем его на енто:

Код

if ($msg['title'])             {              preg_match("/^\s*(re)\s*(?:\(\s*([0-9]+)\s*\))?\s*:/i",$msg['title'],$pmre);              $str=preg_replace("/^(\s*(re)\s*(?:\(\s*([0-9]+)\s*\))?\s*:)+/i", "", $msg['title']);              $old_title=isset($pmre[2])? "Re(".($pmre[2]+1)."):{$str}": (isset($pmre[1])? "Re(2): {$str}": "Re: {$str}");             }else             {              $old_title='Re: ???';             }                 $this->output .= $this->html->Render_msg( array(                                                                      'msg'    => $msg,                                                                      'member' => $member,                                                                      'O_TITLE'   => $old_title,                                                                      'jump'   => $this->jump_html                                                                   )      );

2. Правим skin_msg.php:
- Находим(функция Render_msg($data)):

Код

name='msg_title' value='Re: {$data['msg']['title']}

по идеи... должно один раз встретиться ... т.е. ищем Re:
- и заменяем на:

Код

name='msg_title' value='{$data[O_TITLE]}'

Это сообщение отредактировал(а) Girder - 16.8.2005, 05:04

Поставил на полигне

Поставил на форум