Sanitizing commas from selector

[Orkun]

My Code:

$lpTitle = strip_tags($thisActivity->Location['name']);
$lp = $this->sanitizer->selectorValue($this->pages->get("template=location, title=$lpTitle"));

I get this notice:

Notice: Trying to get property of non-object in /blabla/blabla/blablabla/site/modules/ProcessXMLImport/ProcessXMLImport.module.php on line 247 Notice: 

What I am Doing wrong?

[Ivan Gretsky]

I am not sure that I understand what you are trying to achieve. Why do you want to sanitize $this->pages->get("template=location, title=$lpTitle")? It is not a user input and will return a Page or NullPage anyway. No commas .

If you need to get the id just use $this->pages->get("template=location, title=$lpTitle")->id.

The notice you are getting is from ProcessXMLImport module and is not directly connected with the code you presented.

[Orkun]

Sorry for the slurred declaration of my problem.

It has worked for me this way:

foreach ($Import->xpath('Activities/Activity') as $Activity) {

       $thisActivity = $Activity->ActivityDetail;

       $lpTitle = strip_tags($thisActivity->Location['name']);
					
       $lp = $this->pages->get("template=location, title='$lpTitle'");
       
       if(!$lp->id){
       // if not found create new page
         $lp = new Page();
         ...
       }
}

$lpTitle is a name attribute from the Location node which comes from the xml-feed. And the Problem was that the name attribute had commas inside his value and this had break the selector string. I solve this Problem by wrapping $lpTitle inside apostrophes .

[Martijn Geerts]

From the docs:

$sanitizer->selectorValue($value)

Sanitizes a string that needs to appear in a selector value. Replaces disallowed characters with spaces. If value is not already quoted, it will add quotes if it determines they are necessary (like if it contains commas). It limits the length to 100 characters (multibyte safe).

[LostKobrakai]

To add to Martijn answer. This sanitizer is meant to sanitize a single selector value, not even a whole selector string, e.g.

$mytitlewithcomma = "Hamburg, the great harbour city";

// Will fail
$pages->get("title=$mytitlewithcomma");

// Won't fail
$sanitized = $sanitizer->selectorValue($mytitlewithcomma);
$pages->get("title=$sanitized");