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.
The attached code causes multiple false errors. Was not the case in versions prior to 6.7 RC1.
Created attachment 83284 [details] Mixed HTML/PHP code
Hello, I suspect this could be a duplicate of issue 166460, which was fixed between RC1 and RC2. Could you please check whether this is fixed in NB 6.7 RC2? Thanks.
Well, this is rather an extension of 166460, as there are other false errors too, like "expecting one of <OPTION> or <OPTGROUP>" in the beginning of each <select>, "unmatched tag" near every <option>, and "Unknown attribute type of tag <span>" in lines 167, 171. Cannot check with RC2 anyway as it's not yet available for public download.
BTW you may wish to check also http://www.netbeans.org/issues/show_bug.cgi?id=162492, which I reported earlier. Acually things become worse since, as now not just odds but every <option> gets marked with the "unmatched tag" error.
RC2 is now available here: http://download.netbeans.org/netbeans/6.7/rc2/
Well, then I confirm that RC2 fixed just the inline JS issue, all the rest is still there: see my post at Jun 7 21:17:29.
You are right there are still errors. I tried with NetBeans 6.5.1 and everything works correctly there. I am attaching the screenshot with the errors. Thanks for the report.
Created attachment 83311 [details] Screenshot of errors
This works correctly in 6.5.1 because there are no HTML structure error checks at all. The problem here is that the editor is not able to correctly recover the generated PHP sections and it shows incorrect warnings. There is a hint to disable such checking while the editor behaves unexpectedly as it does in this case. In my opinion it's not P1, because you can easily disable error checking to avoid wrong marks in code and it's probably duplicate of issue #149968 . It is not a regression either since the HTML structure checking is new feature in 6.7
yeah, it looks like a duplicate of issues 149968, 162492 and 160450, but I'd rather Marek evaluated it
Thanks all for your evaluations, it really looks like a problem caused by the issue #149968. BTW I've already fixed the issue in trunk after RC1 cloning. Now (in current trunk) there is only one and correct warning in the attached eir.phtml file. Not a stopper issue since the checks can be disabled per file as jsedek pointed out. I am also not sure if transplanting the fix (if considered as P1) would be safe since I made bigger changes after 6.7 cloning containing the fix and I also saw some regressions in today's trunk build which I am not sure what the cause is.
verified in trunk. Let's deliver the fix of this issue in patch for 6.7
Could you please provide a changeset used for the fix in order that it can be ported to the release67_fixes branch?
hmm, it is basically the change in http://hg.netbeans.org/web-main/diff/2bfa2b356c8a/html.editor.lib/src/org/netbeans/editor/ext/html/parser/SyntaxParser.java but it cannot be that easily isolated I am affraid. It was quite a big change which also fixed this issue as a side effect (I did realized it later). I think I'll have to prepare a patch for the patch #1 clone for you. I'll do it on monday.
I am starting to clone the release67_fixes clone and will prepare the patch today.
The fix extracted from the trunk changeset 2bfa2b356c8a and pushed into release67_fixes clone as 5e078e8dd330. QE please verify the fix since I had to do some additional fixes to discovered problems (which are also in trunk). They were related to tag matching with some templating languages inside.
*** Issue 168368 has been marked as a duplicate of this issue. ***
verified in 6.7.1 NetBeans IDE 6.7.1 RC (Build 200907150227)