Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 289735

Summary: openning php file with <style>@import... cause error msg "Could not open the editor: Argument not valid"
Product: z_Archived Reporter: nir <nir.nahum>
Component: PDTAssignee: PHP Core <php.core-inbox>
Status: CLOSED DUPLICATE QA Contact:
Severity: normal    
Priority: P3 CC: euthanasia_waltz, ganoro
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description nir CLA 2009-09-17 09:42:36 EDT 
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1.3) Gecko/20090824 Firefox/3.5.3
Build Identifier: 20090619-0625

if you try to open a php file with this text in it:
<style>@import"<??>/<??>"</style>
or
<style>@import"<?='x'?>/<?='y'?>"</style>
or
<style>@import"<?='x'?>X<?='y'?>"</style>
and etc
the php file cant be loaded and you get an error msg says
"Could not open the editor: Argument not valid"

if you change style to script or @import to something else
the file will open without an error msg.

Reproducible: Always

Steps to Reproduce:
1. create new php file.
2. put this text in it: <style>@import"<??>/<??>"</style>
3. save it. close it.
4. try to open it.
5. get an error msg. :(



the original line that cause this error:
<style>
@import "<?=$appconfig->appurl?>lib/dojosdk/dijit/themes/<?=$this->dojo_theme?>/<?=$this->dojo_theme?>.css";
</style>
Comment 1 Roy Ganor CLA 2009-09-17 11:40:39 EDT 
The webtools (sse css) guys have already fixed this bug.
See #240309 for more details.

Thanks for reporting!

*** This bug has been marked as a duplicate of bug 240309 ***
Comment 2 nir CLA 2009-09-17 12:11:51 EDT 
im sorry but i dont think this is a duplicate bug,
im using pdt 2.1.0 (relatively new version)
the text the use entered there is working ok in my editor!
but the text i wrote here cause an error msg!

this is a new bug!

try it and you will see.
Comment 3 Toshihiro Izumi CLA 2009-09-18 20:43:01 EDT 
nir,
see Bug 282218.
Comment 4 nir CLA 2009-09-21 03:23:53 EDT 
thank you,
you right, this is a duplicate.

*** This bug has been marked as a duplicate of bug 282218 ***