Backend new media: selecting server clears field of study and date

[st-anton]

When selecting the server, a new set of tabs appear and the fields of study and date are cleared. Naturally, the save fails because of empty mandatory fields.

[tomfuller2]

When editing a study in the backend, if you try to add a new media file, the study and date information do not follow to the window that's opened for the media file.

[tomfuller2]

Also, if you edit a media file from a study - then save it without making changes, the server is not saved. Some corruption is happening.

[bcordis]

@tomfuller2 can we meet on this and see what all needs to be fixed to make it work. Right now i'm having to work through the code and see where this is broken at.

[bcordis]

@st-anton This is a normal function of joomla if you did not save the media before selecting a new server. I have also fixed the new media creation from a message to have a date and message link. I do think there is a better way to select servers.

[tomfuller2]

The problem is I have no idea how to fix this. This is part of the new system you and Eugen developed.

On Sat, May 2, 2015 at 7:56 PM, Brent Cordis notifications@github.com wrote:

@tomfuller2 https://github.com/tomfuller2 can we meet on this and see what all needs to be fixed to make it work. Right now i'm having to work through the code and see where this is broken at.

— Reply to this email directly or view it on GitHub https://github.com/Joomla-Bible-Study/Joomla-Bible-Study/issues/498#issuecomment-98431529 .

[bcordis]

@st-anton @tomfuller2 This should be fixed now.

[st-anton]

Right direction, but still curious behaviour (tested in backend only): Obviously, the last server I saved in the server page is used as type (in my case: legacy server). So I see all tabs and can fill in all the fileds for legacy server. That fits if I have only one server. But: switching over to another type of server has no effect: Tabs are not changing. Also the other server is not saved with the media. It always returns to the 'default' server.

Now also two notices appear, when starting a new media: Notice: Undefined property: stdClass::$study_id in ...\administrator\components\com_biblestudy\views\mediafile\tmpl\edit.php on line 69 Notice: Undefined property: stdClass::$createdate in ...\administrator\components\com_biblestudy\views\mediafile\tmpl\edit.php on line 77

[bcordis]

Ok I'll do some more checking but I'm close now

[bcordis]

@st-anton ok now it saves right but need to find a way to trigger save on change. It still requires that date and study_id have to be selected.

[bcordis]

@st-anton please test.

[st-anton]

Select a study opens the popup for servers, not for studies. The additional tabs do not appear anymore. So none of the parameters can be filled in. Popup Box: Unpublished servers are still selectable; this is misleading to the user. At least the published / unpublished state needs to be displayed.

[bcordis]

@st-anton so what view is not showing study any more?

[bcordis]

Found the error.

[bcordis]

@st-anton retest. This now is fixed. found the error. and added some better search functions.

[st-anton]

Study item and study date now persist. Still no further tabs appear, as the trigger on change does not work.