[Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta

Pat Lockley patrick.lockley at googlemail.com
Fri Apr 20 18:22:28 BST 2012


We need to work out some sort of versioning and release as two of the
fixes I made are of bugs introduced from changes (shouldn't have been
committed)

On Fri, Apr 20, 2012 at 5:58 PM, Ron Mitchell <ronm at mitchellmedia.co.uk> wrote:
> Thanks Pat that's great. I've uploaded your changes and the sharing now
> appears to be working correctly without errors.
>
> Tested switch and that's working too!
>
> Cheers
> Ron
>
> -----Original Message-----
> From: xerte-dev-bounces at lists.nottingham.ac.uk
> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Pat Lockley
> Sent: 20 April 2012 17:43
> To: For Xerte technical developers
> Subject: [Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta
>
> Some one had made preview.php not work - changed a parameter to something it
> wasn't supposed to be. Now fixed.
>
> Also fixed switch.txt and modifed a lot of the file update code (including
> some changes that aren't mine and broke stuff)
>
> On Fri, Apr 20, 2012 at 3:52 PM, Ron Mitchell <ronm at mitchellmedia.co.uk>
> wrote:
>> not in modules/xerte/preview.php but in preview.php in the root I can
>> see there's a require to template_status.php where that function exists.
>>
>>
>>
>> From: xerte-dev-bounces at lists.nottingham.ac.uk
>> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Pat
>> Lockley
>> Sent: 20 April 2012 15:21
>>
>>
>> To: For Xerte technical developers
>> Subject: [Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta
>>
>>
>>
>> Is there an is shares template? Function call
>>
>> On 20 Apr 2012, at 15:09, "Ron Mitchell" <ronm at mitchellmedia.co.uk> wrote:
>>
>> extract
>> //////////////////////////////////////////////////////////////////////
>> /////////////////////
>>
>> function show_preview_code($row, $row_username){
>>
>>
>>
>>                global $xerte_toolkits_site;
>>
>>
>>
>>                /*
>>
>>                * Format the XML strings to provide data to the engine
>>
>>                */
>>
>>
>>
>>                if(!file_exists($xerte_toolkits_site->users_file_area_short
> .
>> $row['template_id'] . "-" . $row_username['username'] . "-" .
>> $row['template_name'] . "/preview.xml")){
>>
>>
>>
>>                               $buffer =
>> file_get_contents($xerte_toolkits_site->users_file_area_short .
>> $row['template_id'] . "-" . $row_username['username'] . "-" .
>> $row['template_name'] . "/data.xml");
>>
>>
>>
>>                               $fp =
>> fopen($xerte_toolkits_site->users_file_area_short . $row['template_id'] .
>> "-" . $row_username['username'] . "-" . $row['template_name'] .
>> "/preview.xml","x");
>>
>>                               fwrite($fp, $buffer);
>>
>>                               fclose($fp);
>>
>>
>>
>>                }
>>
>>
>>
>>                $string_for_flash_xml =
>> $xerte_toolkits_site->users_file_area_short . $row['template_id'] . "-" .
>> $row_username['username'] . "-" . $row['template_name'] . "/preview.xml" .
>> "?time=" . time();
>>
>>
>>
>>                $string_for_flash =
>> $xerte_toolkits_site->users_file_area_short . $row['template_id'] . "-" .
>> $row_username['username'] . "-" . $row['template_name'] . "/";
>>
>> extract
>> //////////////////////////////////////////////////////////////////////
>> /////////////////////
>>
>>
>>
>>
>>
>>
>>
>> From: xerte-dev-bounces at lists.nottingham.ac.uk
>> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Pat
>> Lockley
>> Sent: 20 April 2012 15:04
>> To: For Xerte technical developers
>> Subject: [Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta
>>
>>
>>
>> Not on pc
>>
>>
>>
>> What is the path variable made from?
>>
>> On 20 Apr 2012, at 14:58, "Ron Mitchell" <ronm at mitchellmedia.co.uk> wrote:
>>
>> Assuming that you mean modules/xerte/preview.php see attached.
>>
>>
>>
>> Presumably somewhere $row_username['username'] get's passed the owner
>> ID when it's a shared project?
>>
>>
>>
>> From: xerte-dev-bounces at lists.nottingham.ac.uk
>> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Pat
>> Lockley
>> Sent: 20 April 2012 14:47
>> To: For Xerte technical developers
>> Subject: [Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta
>>
>>
>>
>> No, what is in the code
>>
>> On 20 Apr 2012, at 14:38, "Ron Mitchell" <ronm at mitchellmedia.co.uk> wrote:
>>
>> Do you mean what happens when clicking preview?
>>
>> The error below
>>
>>
>>
>> Warning:
>> file_get_contents(USER-FILES/31-patrick.lockley at googlemail.com-Notting
>> ham/da
>> ta.xml): failed to open stream: No such file or directory in
>> /var/www/vle/xot/modules/xerte/preview.php on line 35 Warning:
>>
> fopen(USER-FILES/31-patrick.lockley at googlemail.com-Nottingham/preview.xml):
>> failed to open stream: No such file or directory in
>> /var/www/vle/xot/modules/xerte/preview.php on line 37 Warning:
>> fwrite() expects parameter 1 to be resource, boolean given in
>> /var/www/vle/xot/modules/xerte/preview.php on line 38 Warning:
>> fclose() expects parameter 1 to be resource, boolean given in
>> /var/www/vle/xot/modules/xerte/preview.php on line 39
>>
>>
>>
>>
>> Followed by the preloader trying to load the xml but obviously not
>> finding it.
>>
>>
>>
>> From: xerte-dev-bounces at lists.nottingham.ac.uk
>> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Pat
>> Lockley
>> Sent: 20 April 2012 14:21
>> To: For Xerte technical developers
>> Subject: [Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta
>>
>>
>>
>> What does modules/preview.php show
>>
>> On 20 Apr 2012, at 14:08, "Ron Mitchell" <ronm at mitchellmedia.co.uk> wrote:
>>
>> You are user 10!
>>
>>
>>
>> It seems like when the shared editor (in this case you) tries to
>> preview the LO the code is looking in your user-files folder rather
>> than the owner's e.g. mine.
>>
>>
>>
>> Whereas in previous versions it correctly looks in the owner's folder.
>>
>>
>>
>> From: xerte-dev-bounces at lists.nottingham.ac.uk
>> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Pat
>> Lockley
>> Sent: 20 April 2012 13:52
>> To: For Xerte technical developers
>> Subject: [Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta
>>
>>
>>
>> No, that. Who is user 10?
>>
>> On 20 Apr 2012, at 12:11, "Ron Mitchell" <ronm at mitchellmedia.co.uk> wrote:
>>
>> Do you mean the following?
>>
>>
>>
>> Click here to add this user
>>
>>
>>
>> javascript:share_this_template('31', '10')
>>
>>
>>
>> or somewhere else?
>>
>>
>>
>> From: xerte-dev-bounces at lists.nottingham.ac.uk
>> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Pat
>> Lockley
>> Sent: 20 April 2012 11:55
>> To: For Xerte technical developers
>> Subject: [Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta
>>
>>
>>
>> Hover over the share link and note the parameters?
>>
>> On 20 Apr 2012, at 11:42, "Ron Mitchell" <ronm at mitchellmedia.co.uk> wrote:
>>
>> Hi Pat
>>
>> apologies for the delayed response but I wanted to test this on a pre
>> 1.8 install to verify what happens...
>>
>>
>>
>> In a pre 1.8 installation if I share an LO with another user when that
>> user clicks preview it loads preview.xml from my lo folder e.g.
>> USER-FILES/31-ronm-Nottingham/preview.xml?time=1334918728
>>
>>
>>
>> In this current 1.8 beta if I share an LO with your account and then
>> you try to preview that LO it's looking at
>> USER-FILES/31-patrick.lockley at googlemail.com-Nottingham/preview.xml?ti
>> me=1334918794
>>
>>
>>
>> So unless the way this works has changed (?) it seems like the preview
>> code is looking in the wrong place e.g. the shared user account rather
>> than the owner account. In the example below
>> USER-FILES/31-ronm-Nottingham/preview.xml exists but
>> USER-FILES/31-patrick.lockley at googlemail.com-Nottingham/preview.xml
>> doesn't but I'm not sure it should? Is the wrong username being passed
> somewhere?
>>
>>
>>
>> Cheers
>>
>> Ron
>>
>>
>>
>> From: xerte-dev-bounces at lists.nottingham.ac.uk
>> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Pat
>> Lockley
>> Sent: 20 April 2012 09:44
>> To: For Xerte technical developers
>> Subject: [Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta
>>
>>
>>
>> Is the file there?
>>
>> On Friday, April 20, 2012, Ron Mitchell wrote:
>>
>> Hi Pat
>> it seems I spoke to soon when saying the sharing functionality is
>> fixed :-( It is fixed in terms of owners are now able to share access
>> with other users but it seems those other users get an error when trying
> to preview the LO.
>> I've shared a very simple LO with your account (just has a title page)
>> and you'll see that when you try to preview you'll get the following
> errors:
>>
>> Warning:
>> file_get_contents(USER-FILES/31-patrick.lockley at googlemail.com-Notting
>> ham/da
>> ta.xml): failed to open stream: No such file or directory in
>> /var/www/vle/xot/modules/xerte/preview.php on line 35 Warning:
>>
> fopen(USER-FILES/31-patrick.lockley at googlemail.com-Nottingham/preview.xml):
>> failed to open stream: No such file or directory in
>> /var/www/vle/xot/modules/xerte/preview.php on line 37 Warning:
>> fwrite() expects parameter 1 to be resource, boolean given in
>> /var/www/vle/xot/modules/xerte/preview.php on line 38 Warning:
>> fclose() expects parameter 1 to be resource, boolean given in
>> /var/www/vle/xot/modules/xerte/preview.php on line 39
>>
>> Jonathan also reported the same issue with an LO I shared with quite a
>> few accounts and obviously this hasn't been a problem with previous
>> versions. Is it another mangle as a result of the svn merge? Is there a
> fix?
>>
>> Cheers
>> Ron
>>
>> -----Original Message-----
>> From: xerte-dev-bounces at lists.nottingham.ac.uk
>> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Ron
>> Mitchell
>> Sent: 19 April 2012 19:59
>> To: 'For Xerte technical developers'
>> Subject: [Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta
>>
>> Hi Pat
>> thanks for the code updates and regarding convention with the fields
>> feel free to change the category column or let me know what you feel
>> they should be.
>>
>> I've uploaded your code updates and the project sharing is definitely
>> fixed now but I can't verify the management page fixes because the
>> management.php page is now showing a blank screen? Any ideas on that?
>>
>> Ron
>>
>> -----Original Message-----
>> From: xerte-dev-bounces at lists.nottingham.ac.uk
>> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Pat
>> Lockley
>> Sent: 19 April 2012 19:18
>> To: For Xerte technical developers
>> Subject: [Xerte-dev] Re: testing and tracking issues with XOT 1.8 beta
>>
>> Could we get some convention with the fields - some stuff is PHP, some
>> stuff is referencing a page - it's a bit messed up.
>>
>> On Thu, Apr 19, 2012 at 5:31 PM, Ron Mitchell
>> <ronm at mitchellmedia.co.uk>
>> wrote:
>>> Hi all
>>> thanks to those who have been testing the XOT 1.8 beta via the JISC
>>> Techdis installation and recording issues via the Google doc we have
>>> set
>> up.
>>>
>>> The list of items in the Google doc has increased dramatically since
>>> I shared that this morning but for obvious reasons please don't share
>>> the link to that via this list. If anyone else on the dev list
>>> reading this is keen to get involved in testing and better still
>>> fixing please contact me off-list to send you details.
>>>
>>> The Google doc is just an initial way of sharing test results and I
>>> guess key bugs will be transferred to the google code issue tracker.
>>>
>>> The categories of issues have grown but for me there are some
>>> immediate issues categorised as php/website code issues e.g. can't
>>> share a project, can't save anything via the management page and
>>> lot's of errors on various management pages.
>>>
>>> These and more are listed in the Google doc but I'm not sure when,
>>> how and by whom the various issues will be fixed. There is a column
>>> for who will fix what and some additions to that but it seems there's
>>> quite a way to go before a 1.8 release is likely to be ready. However
>>> I think the Google doc is working as a way of listing and commenting
>>> on the issues so would be great if we can continue with that over the
>>> next
>> few days.
>>>
>>> Cheers
>>> Ron
>>>
>>>
>>> _______________________________________________
>>> Xerte-dev mailing list
>>> Xerte-dev at lists.nottingham.ac.uk
>>> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>>>
>>> This message and any attachment are intended solely for the addressee
>>> and
>> may contain confidential information. If you have received this
>> message in error, please send it back to me, and immediately delete
>> it.   Please do not use, copy or disclose the information contained in
>> this message or in any attachment.  Any views or opinions expressed by
>> the author of this email do not necessarily reflect the views of the
> University of Nottingham.
>>>
>>> This message has been checked for viruses but the contents of an
>>> attachment may still contain software viruses which could damage your
>> computer system:
>>> you are advised to perform your own checks. Email communications with
>>> the University of Nottingham may be monitored as permitted by UK
>> legislation.
>>>
>>
>> _______________________________________________
>> Xerte-dev mailing list
>> Xerte-dev at lists.nottingham.ac.uk
>> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>> This message and any attachment are intended solely for the addressee
>> and may contain confidential information. If you have received this
>> message in error, please s
>>
>>
>>
>> This message and any attachment are intended solely for the addressee
>> and may contain confidential information. If you have received this
>> message in error, please send it back to me, and immediately delete
>> it. Please do not use, copy or disclose the information contained in
>> this message or in any attachment. Any views or opinions expressed by
>> the author of this email do not necessarily reflect the views of the
> University of Nottingham.
>>
>> This message has been checked for viruses but the contents of an
>> attachment may still contain software viruses which could damage your
> computer system:
>> you are advised to perform your own checks. Email communications with
>> the University of Nottingham may be monitored as permitted by UK
> legislation.
>>
>> _______________________________________________
>> Xerte-dev mailing list
>> Xerte-dev at lists.nottingham.ac.uk
>> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>>
>> This message and any attachment are intended solely for the addressee
>> and may contain confidential information. If you have received this
>> message in error, please send it back to me, and immediately delete
>> it.   Please do not use, copy or disclose the information contained in
>> this message or in any attachment.  Any views or opinions expressed by
>> the author of this email do not necessarily reflect the views of the
> University of Nottingham.
>>
>> This message has been checked for viruses but the contents of an
>> attachment may still contain software viruses which could damage your
> computer system:
>> you are advised to perform your own checks. Email communications with
>> the University of Nottingham may be monitored as permitted by UK
> legislation.
>>
>> _______________________________________________
>> Xerte-dev mailing list
>> Xerte-dev at lists.nottingham.ac.uk
>> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>>
>> This message and any attachment are intended solely for the addressee
>> and may contain confidential information. If you have received this
>> message in error, please send it back to me, and immediately delete
>> it.   Please do not use, copy or disclose the information contained in
>> this message or in any attachment.  Any views or opinions expressed by
>> the author of this email do not necessarily reflect the views of the
> University of Nottingham.
>>
>> This message has been checked for viruses but the contents of an
>> attachment may still contain software viruses which could damage your
> computer system:
>> you are advised to perform your own checks. Email communications with
>> the University of Nottingham may be monitored as permitted by UK
> legislation.
>>
>> _______________________________________________
>> Xerte-dev mailing list
>> Xerte-dev at lists.nottingham.ac.uk
>> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>>
>> This message and any attachment are intended solely for the addressee
>> and may contain confidential information. If you have received this
>> message in error, please send it back to me, and immediately delete
>> it.   Please do not use, copy or disclose the information contained in
>> this message or in any attachment.  Any views or opinions expressed by
>> the author of this email do not necessarily reflect the views of the
> University of Nottingham.
>>
>> This message has been checked for viruses but the contents of an
>> attachment may still contain software viruses which could damage your
> computer system:
>> you are advised to perform your own checks. Email communications with
>> the University of Nottingham may be monitored as permitted by UK
> legislation.
>>
>> _______________________________________________
>> Xerte-dev mailing list
>> Xerte-dev at lists.nottingham.ac.uk
>> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>>
>> This message and any attachment are intended solely for the addressee
>> and may contain confidential information. If you have received this
>> message in error, please send it back to me, and immediately delete
>> it.   Please do not use, copy or disclose the information contained in
>> this message or in any attachment.  Any views or opinions expressed by
>> the author of this email do not necessarily reflect the views of the
> University of Nottingham.
>>
>> This message has been checked for viruses but the contents of an
>> attachment may still contain software viruses which could damage your
> computer system:
>> you are advised to perform your own checks. Email communications with
>> the University of Nottingham may be monitored as permitted by UK
> legislation.
>>
>> <preview.txt>
>>
>> _______________________________________________
>> Xerte-dev mailing list
>> Xerte-dev at lists.nottingham.ac.uk
>> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>>
>> This message and any attachment are intended solely for the addressee
>> and may contain confidential information. If you have received this
>> message in error, please send it back to me, and immediately delete
>> it.   Please do not use, copy or disclose the information contained in
>> this message or in any attachment.  Any views or opinions expressed by
>> the author of this email do not necessarily reflect the views of the
> University of Nottingham.
>>
>> This message has been checked for viruses but the contents of an
>> attachment may still contain software viruses which could damage your
> computer system:
>> you are advised to perform your own checks. Email communications with
>> the University of Nottingham may be monitored as permitted by UK
> legislation.
>>
>> _______________________________________________
>> Xerte-dev mailing list
>> Xerte-dev at lists.nottingham.ac.uk
>> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>>
>> This message and any attachment are intended solely for the addressee
>> and may contain confidential information. If you have received this
>> message in error, please send it back to me, and immediately delete
>> it.   Please do not use, copy or disclose the information contained in
>> this message or in any attachment.  Any views or opinions expressed by
>> the author of this email do not necessarily reflect the views of the
> University of Nottingham.
>>
>> This message has been checked for viruses but the contents of an
>> attachment may still contain software viruses which could damage your
> computer system:
>> you are advised to perform your own checks. Email communications with
>> the University of Nottingham may be monitored as permitted by UK
> legislation.
>>
>>
>> _______________________________________________
>> Xerte-dev mailing list
>> Xerte-dev at lists.nottingham.ac.uk
>> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>>
>> This message and any attachment are intended solely for the addressee
>> and may contain confidential information. If you have received this
>> message in error, please send it back to me, and immediately delete
>> it.   Please do not use, copy or disclose the information contained in
>> this message or in any attachment.  Any views or opinions expressed by
>> the author of this email do not necessarily reflect the views of the
> University of Nottingham.
>>
>> This message has been checked for viruses but the contents of an
>> attachment may still contain software viruses which could damage your
> computer system:
>> you are advised to perform your own checks. Email communications with
>> the University of Nottingham may be monitored as permitted by UK
> legislation.
>>
>>
>
> _______________________________________________
> Xerte-dev mailing list
> Xerte-dev at lists.nottingham.ac.uk
> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev
>
>
> _______________________________________________
> Xerte-dev mailing list
> Xerte-dev at lists.nottingham.ac.uk
> http://lists.nottingham.ac.uk/mailman/listinfo/xerte-dev



More information about the Xerte-dev mailing list