php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #61979 LibXMLError Object property 'message' contains LF at the end of string
Submitted: 2012-05-09 09:23 UTC Modified: 2021-03-16 16:04 UTC
From: piotr at smartwebstudio dot cz Assigned: cmb (profile)
Status: Not a bug Package: *XML functions
PHP Version: 5.3.13 OS: Windows Vista 32bit
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: piotr at smartwebstudio dot cz
New email:
PHP Version: OS:

 

 [2012-05-09 09:23 UTC] piotr at smartwebstudio dot cz
Description:
------------
---
From manual page: http://www.php.net/manual/pl/function.libxml-get-last-error.php
---
Tested on PHP 5.3.0, Apache 2.2.11

LibXMLError Object property 'message' contains LF (ASCII 10) at the end of 
string.
Makes it harder to compare error messages that should be ignored or assign 
translation strings.

Test script:
---------------
libxml_use_internal_errors(true);
// Define xml string that contains known error
$vcard = new SimpleXMLElement("<vCard xmlns='vcard-temp'></vcard>");

// Retrieve last LibXMLError object
$last_error = libxml_get_last_error();

// Should be true
var_dump($last_error->message == "xmlns: URI vcard-temp is not absolute");

// Workaround
var_dump(trim($last_error->message) == "xmlns: URI vcard-temp is not absolute");

Expected result:
----------------
bool(true)
bool(true)

Actual result:
--------------
bool(false)
bool(true)

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2016-06-13 18:14 UTC] cmb@php.net
In my opinion this is not a bug, as I understand the error message
as "human-readable informative error message"[1]. And I would
rather check for the error code to avoid issues due to libxml
changing the error messages or localization issues.

[1] <http://www.xmlsoft.org/html/libxml-xmlerror.html#xmlError>
 [2018-03-14 23:10 UTC] cmb@php.net
-Package: XML related +Package: *XML functions
 [2021-03-16 16:04 UTC] cmb@php.net
-Status: Open +Status: Not a bug -Assigned To: +Assigned To: cmb
 [2021-03-16 16:04 UTC] cmb@php.net
Well, the LFs are indeed provided by libxml2 already[1], and it
makes no sense to strip them out.

[1] <https://github.com/GNOME/libxml2/blob/868e49cffdbfddeab0d1c391e0ce7f4f98ea59a1/parser.c#L9289>
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Sat Dec 21 17:01:58 2024 UTC