PHP class: Unable to access array in another function

Question

I tried a lot of search but unable to figure out why array $wordlinks in function DoWordLink is not carrying values from function __construct. PHP class code as below:

<?php

class autolinkkeyword
{

    public $wordlinks = array();

    public function __construct(){
        $query = mysql_query("SELECT keyword FROM library");
                while ($row = mysql_fetch_array($query))
                {
                    $this->wordlinks [$row["keyword"]] = $row["keyword"];
                }
    }               

    public function linkkeywords ($posts)
    {                
            function DoWordLink($match)
            {
                $rpl=$match[1];
                if(isset($wordlinks[$rpl])) 
                {
                        $kword = $this->wordlinks[$rpl];
                        $rpl="<a class=\"keyword_link\" href=\"#\" onclick=\"popup('popUpDiv'); 
                                ajax_loadContent('kword', 'library.php?keyword=$kword')\">$kword</a>";
                        unset($this->wordlinks[$match[1]]);
                }
                return $rpl;
            }

            $wl=array_keys($this->wordlinks);           
            $pm="/((?<=\s|^)(?:" . implode('|',$wl) .")(?=\.|\!|\?|\,|\'|\s|$))/im";
            foreach($posts as $key => $mainbody)
            {
                $mainbody=preg_replace_callback($pm, 'DoWordLink', $mainbody)  ;    
                echo $mainbody;
            }

    }       
}
?>

isset($wordlinks[$rpl]) should be isset($this->wordlinks[$rpl]) — thetaiko
– thetaiko, Commented Jun 21, 2011 at 20:51

saber · Accepted Answer · 2011-06-22 02:10:20Z

1

You can make it an actual method of that class and call it using this method: http://www.php.net/manual/en/language.pseudo-types.php#language.types.callback like:

preg_replace_callback($pm, array($this, 'DoWordLink'), $mainbody);

Change DoWordLink function so it is part of the class like:

class autolinkkeyword
{
  function DoWordLink($match)
  {
    $rpl=$match[1];
    if(isset($this->wordlinks[$rpl])) 
    {
      $kword = $this->wordlinks[$rpl];
      $rpl="<a class=\"keyword_link\" href=\"#\" onclick=\"popup('popUpDiv'); 
      ajax_loadContent('kword', 'library.php?keyword=$kword')\">$kword</a>";
      unset($this->wordlinks[$match[1]]);
    }
    return $rpl;
  }
}

edited Jun 22, 2011 at 2:10

saber

53 bronze badges

answered Jun 21, 2011 at 21:04

Justin

2071 silver badge6 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

r3c · Accepted Answer · 2011-06-21 20:52:41Z

1

aren't you missing a "this->" construct here? if(isset($this->wordlinks[$rpl]))

answered Jun 21, 2011 at 20:52

r3c

4983 silver badges8 bronze badges

Comments

Gustav · Accepted Answer · 2011-06-21 20:53:43Z

0

Use the $this everywhere you refer to $wordlinks.

$this->wordlinks

answered Jun 21, 2011 at 20:53

Gustav

1,3411 gold badge12 silver badges24 bronze badges

Comments

Fidi · Accepted Answer · 2011-06-21 20:54:59Z

0

You need to access the property in your linkkeywords-method with the object-accessor, too!

public function linkkeywords ($posts)
{

   // Here use $this->wordlinks not $wordlinks
}

answered Jun 21, 2011 at 20:54

Fidi

5,8241 gold badge20 silver badges25 bronze badges

Collectives™ on Stack Overflow

PHP class: Unable to access array in another function

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related