Tag bbcode [member] - nowe opcje

[Pavulon]

Dzięki niewielkim modyfikacją możemy sprawić że tag [ member] stanie się bardziej przyjazny.

Standardowo używany jest tak:

[member='DawPi']

a efekt mamy taki: @DawPi

Docelowo można go będzie używać tak jak poprzednio lub podając id(co jest wg mnie bezpieczniejsze):

[member='1']

a w efekcie otrzymamy link taki jak poprzednio z tym wyjątkiem że będzie miał kolor(styl) grupy do której należy użytkownik. DawPi

Edytujemy plik:

admin/sources/classes/bbcode/custom/defaults.php

Znajdujemy:

					if( $member['members_display_name'] )
				{
					$existing[ $_option ]	= array( 'member_id' => $member['member_id'], 'members_display_name' => $member['members_display_name'] );
					$this->cache->updateCacheWithoutSaving( '_tmp_bbcode_members', $existing );

					$_content = $this->_buildOutput( $existing[ $_option ] );
				}

Zamieniamy na:

					if( $member['members_display_name'] )
				{
					$name = IPSLib::makeNameFormatted($member['members_display_name'], $member['member_group_id']);

					$existing[ $_option ]	= array( 'member_id' => $member['member_id'], 'members_display_name' => $name );
					$this->cache->updateCacheWithoutSaving( '_tmp_bbcode_members', $existing );

					$_content = $this->_buildOutput( $existing[ $_option ] );
				} else if (intval($_option) == $_option) {
					$member = IPSMember::load( $_option );

					if( $member['members_display_name'] )
					{
						$name = IPSLib::makeNameFormatted($member['members_display_name'], $member['member_group_id']);

						$existing[ $_option ]	= array( 'member_id' => $member['member_id'], 'members_display_name' => $name );
						$this->cache->updateCacheWithoutSaving( '_tmp_bbcode_members', $existing );

						$_content = $this->_buildOutput( $existing[ $_option ] );
					}
				}

[Gość Komar]

w tym kodzie czym zamieniamy, jest o jeden znak } za duzo (na koncu) albo w tym CO zamieniamy jest o jeden znak } za malo xD

[Pavulon]

w tym kodzie czym zamieniamy, jest o jeden znak } za duzo (na koncu) albo w tym CO zamieniamy jest o jeden znak } za malo xD

Tak, o jedną klamerkę za dużo skopiowałem w kodzie na który zamieniamy.

[Gość Komar]

Tak, o jedną klamerkę za dużo skopiowałem w kodzie na który zamieniamy.

@up:to od razu mozesz poprawic xD

[DawPi]

Poprawione.

[unbreak]

Co jeśli ktoś ma w nicku tylko cyfrę? np 100

Skrypt wrzuci link do osoby o tym nicku czy o takim id? :>

[Pavulon]

Co jeśli ktoś ma w nicku tylko cyfrę? np 100

Skrypt wrzuci link do osoby o tym nicku czy o takim id? :>

Najpierw sprawdzany jest nick. Jeżeli znajdzie się user o nick'u "100" to on zostanie wyświetlony, jeżeli nie to user o id "100"

[unbreak]

Dzięki za rozwianie wątpliwości, fajny pomysł

[DawPi]

Co jeśli będzie i user o ID 100 i użytkownik o takiej nazwie?

Zmień to lepiej:

if( $member['members_display_name'] )

Stosując funkcję is_string().

[Pavulon]

z tym is_string() może nie przejść bo $_option (nick lub id) zawsze przekazywane jest jako string.

więc tak is_string("100") jak i is_string("DawPi") da TRUE.

można by spróbować regexp'em albo np:

intval($_option) != $_option

[DawPi]

Można.

[Pavulon]

Dziwne, teoretycznie powinno działać to co pisałem wcześnie ale praktycznie nie chciało.

Na szczęście była jeszcze opcja is_numeric()

Czyli kod będzie taki:

					if( $member['members_display_name'] && !is_numeric($_option) )
				{
					$name = IPSLib::makeNameFormatted($member['members_display_name'], $member['member_group_id']);

					$existing[ $_option ]	= array( 'member_id' => $member['member_id'], 'members_display_name' => $name );
					$this->cache->updateCacheWithoutSaving( '_tmp_bbcode_members', $existing );

					$_content = $this->_buildOutput( $existing[ $_option ] );
				} else if ( is_numeric($_option) ) {
					$member = IPSMember::load( $_option );

					if( $member['members_display_name'] )
					{
						$name = IPSLib::makeNameFormatted($member['members_display_name'], $member['member_group_id']);

						$existing[ $_option ]	= array( 'member_id' => $member['member_id'], 'members_display_name' => $name );
						$this->cache->updateCacheWithoutSaving( '_tmp_bbcode_members', $existing );

						$_content = $this->_buildOutput( $existing[ $_option ] );
					}
				}

Byłbym wdzięczny jak byś mógł poprawić pierwszy post

[unbreak]

Najs.

Swoją drogą to nie rozwiązuje problemu:

Jest user o nicku 100

Jest inny user który mi id==100,

chcemy wyświetlić tego drugiego, niestety zawsze wyświetlać nam będzie tego pierwszego Tak, wiem, wtedy można zastosować jego nick

Weźmy skrajny przypadek...

user o nicku 100 ma id 200

user o nicku 200 ma id 100

Sprawa będzie trochę pomieszana ;p

dla twórcy Fajna sprawa

[Pavulon]

no niestety, ale nie ma co też tworzyć całkiem nowych tagów [ memberid=dddd] lub modyfikować aktualne dla takich skrajnych przypadków.

zawsze można by próbować modyfikacji gdzie do numeru id trzeba by dodawać id [ member='id100'] ( ale wtedy co będzie jak ktoś będzie miał nick "id100")

[+matius]

a może 2 osobne bbcode ? jeden na same nazwy a drugi na same ID np [ memberID=100' ] ?

[DawPi]

Możecie mi napisać, po co Wam te wszystkie cuda na kiju?

[MafiaDL]

[/url] Edytujemy plik:

admin/sources/classes/bbcode/custom/defaults.php

Znajdujemy:

 if( $member['members_display_name'] ) { $existing[ $_option ] = array( 'member_id' => $member['member_id'], 'members_display_name' => $member['members_display_name'] ); $this->cache->updateCacheWithoutSaving( '_tmp_bbcode_members', $existing ); $_content = $this->_buildOutput( $existing[ $_option ] ); }

Dziwnie u mnie jest w innej kolejności:

$existing[ $_option ] = array( 'member_id' => $member['member_id'], 'members_display_name' => $member['members_display_name'], 'members_seo_name' => $member['members_seo_name'] );<br>$this->cache->updateCacheWithoutSaving( '_tmp_bbcode_members', $existing );<br><br>if( $member['members_display_name'] )<br>{<br>$_content = $this->_buildOutput( $existing[ $_option ] );

Coś takiego

[MafiaDL]

i nie działa

[Pavulon]

i nie działa

Może dlatego, że temat był aktualny 2 lata temu i pod nową wersję trzeba by go przystosować?

[MafiaDL]

A może Myślałem że tylko u mnie jest inaczej Wiec nie ma sposobu na kolor : P

[Pavulon]

Wiec nie ma sposobu na kolor : P

Więc niech tak ci będzie...

[Greedy]

IPB 3.1.4 - Brak tego kodu w defaults.php.