Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [pdt-dev] Spellcheck patch

Hi,
 
and one more patch what fixes broken spellchecking around php code (i.e. in html content).
So,  the bug 322808 can be closed now.
 
Best regards,
Iljas
 
16.05.2012, 10:14, "赵忠伟" <zhaozhongwei@xxxxxxxxxxxxxxxx>:
hi,
I have committed the patch,thanks very much!

On Sun, Mar 25, 2012 at 3:05 AM, Iljas <ilj-ilj@xxxxxxxxx> wrote:
Hello!

I suggest the patch, which enables spellchecking in strings and comments of the php content.

Best regards,
Iljas
_______________________________________________
pdt-dev mailing list
pdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/pdt-dev


--

Thanks!

Best Regards!

Zhao

_______________________________________________
pdt-dev mailing list
pdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/pdt-dev

--
Iljas
diff --git a/plugins/org.eclipse.php.ui/src/org/eclipse/php/internal/ui/spelling/SpellcheckDelegateAdapterFactory.java b/plugins/org.eclipse.php.ui/src/org/eclipse/php/internal/ui/spelling/SpellcheckDelegateAdapterFactory.java
index adf4b14..0c739bd 100644
--- a/plugins/org.eclipse.php.ui/src/org/eclipse/php/internal/ui/spelling/SpellcheckDelegateAdapterFactory.java
+++ b/plugins/org.eclipse.php.ui/src/org/eclipse/php/internal/ui/spelling/SpellcheckDelegateAdapterFactory.java
@@ -82,6 +82,17 @@ public class SpellcheckDelegateAdapterFactory implements IAdapterFactory {
 
 			}
 
+			if (!shouldSpellcheck) {
+				IAdapterFactory parentDelegateFactory = new org.eclipse.wst.xml.ui.internal.spelling.SpellcheckDelegateAdapterFactory();
+				Object delegate = parentDelegateFactory.getAdapter(model,
+						ISpellcheckDelegate.class);
+				if (delegate instanceof ISpellcheckDelegate) {
+					ISpellcheckDelegate spellDelegate = (ISpellcheckDelegate) delegate;
+					shouldSpellcheck = spellDelegate.shouldSpellcheck(offset,
+							model);
+				}
+			}
+
 			return shouldSpellcheck;
 		}
 

Back to the top