Hi Kang Jingchuan,

See my comments inline:

kangjch wrote:
Hi All,

About this issue78095, by my debugging, I find that the default(directly
"Ctrl+x") pasting

in this case is " Html Formats without
format"(SOT_FORMATSTR_ID_HTML_NO_COMMENT == 123)

Following the reproducing steps, below functions was called.
int SwTransferable::_PasteFileContent(.) sw\source\dochdl\swdtflvr.cxx

There's a ...\ui\... missing from the path. (Also, if you have the information easily available, it is always good to add the file's CVS revision number, like .../swdtflvr.cxx:1.107. Emacs, for example, typically shows the revision number of an open file in the status line.)

{

default:

         if( rData.GetSotStorageStream( nFmt, xStrm ) )

         {

              if( ( SOT_FORMATSTR_ID_HTML_SIMPLE == nFmt ) ||

                ( SOT_FORMATSTR_ID_HTML_NO_COMMENT == nFmt ) )

              {

//kangjch  added

                   if( rData.GetSotStorageStream( SOT_FORMAT_RTF, xStrm ) )

That's cheating, I would say: The user requests to insert HTML, but you change that to RTF instead. (Plus, it does not work if there is no RTF available for the clipboard data.) I guess the real fix would be to the ReadHTML function instead...

                   {

                       pStream = &xStrm;

                       pRead = ReadRtf;

                   }

//kangjch  added

                   else

                   {

                       pStream = aMSE40ClpObj.IsValid( *xStrm );

                       pRead = ReadHTML;

                       pRead->SetReadUTF8( TRUE );

                       //pRead->SetBaseURL( aMSE40ClpObj.GetBaseURL() );

                       BOOL bNoComments =

                            ( nFmt == SOT_FORMATSTR_ID_HTML_NO_COMMENT );

                       pRead->SetIgnoreHTMLComments( bNoComments );

                   }

              }

              ..

              ..

}

By adding "//kangjch  added"'s codes, the issue can be fixed, but, exactly I
know these classes only a little,
I'm afraid of this modification must have risks. Would you like give me some
suggestion?

Whether does my adding codes have risk or not?

@Stephan: by my investigations, I find that, in SRC680_m215, the file is
also updated from 1.07 to 1.08,

"cvs log swdtflvr.cxx" and "cvs diff -r1.107 -r1.108 swdtflvr.cxx" show that those changes are most probably not relevant to your problem, so it should be safe to stay with your current versoin (SRC680m210?).

And only "Html format" and "Html whitout format" can not work.

Sorry, what do you mean with the above sentence?

Whether do I
need to updated my work area into m215 or not?

-Stephan

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to