This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 120201 - PHP HTML code combination: Highlighting of paired tags is broken
Summary: PHP HTML code combination: Highlighting of paired tags is broken
Status: RESOLVED FIXED
Alias: None
Product: php
Classification: Unclassified
Component: Editor (show other bugs)
Version: 6.x
Hardware: All All
: P2 blocker (vote)
Assignee: Denis Anisimov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-10-26 16:15 UTC by kozlov
Modified: 2008-03-27 11:31 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description kozlov 2007-10-26 16:15:52 UTC
Build071018.

Steps to reproduce:

- create php file;

<html>
<body>

<?php
/**
 * PHP Template.
 */

?>
 
</body>
</html>

- select <body> tag.

Close </body> tag is not highlighted.
Comment 1 Denis Anisimov 2007-11-06 17:39:13 UTC
Bug is invalid.

In described case "<body>" and "</body>" could be unpaired tags.
Only after analyze ( or execution ) of php part we can know result output.
So user can put paired "</body>" tag inside   PHP block ( it can be outed via echo "</body>" ).
So there is no possibility to know about pairing tags that separated by PHP blocks.
Only unseparated tags should be highlighted.
Comment 2 kozlov 2007-12-04 12:49:28 UTC
If we can find pairs in HTML we should highlight them.
Comment 3 Denis Anisimov 2007-12-17 06:22:25 UTC
Please provide exact meaning of your sentence :
"If we can find pairs in HTML we should highlight them."
Who can find pairs ? 

Could you please point to me the pair for "<a>" tag in the following two examples:
First :
<a>
<?php 
if ( $a ) {
     echo "</a>";
}
?>
<br>
</a>

Second:
<a>
<?php 
echo "link1";
echo "</a>";
echo "<a>link2</a>";
?>
</a>
Comment 4 Petr Pisl 2008-03-27 11:10:27 UTC
I'm closing this issue as fixed, because now the tag matching works in the different way. On the other hand now there is
different problem. See issue #131275