I guess value for the new property is saved in DB properly? If yes, please check client debug window and ensure new value is available inside any copy of roomObject (or currentRoomObject or something like that), you click on object in debug window to see all its properties
On Sun, Oct 11, 2015 at 9:43 PM, Susheel Jalali <[email protected]> wrote: > Dear Maxim, > > Following your guidance, after adding the entry in RoomsPanel.html, we are > able to see the new Checkbox in the Rooms Layout of Admin => Conference > room configuration. > > However, even after CHECK and SAVE of this checkbox, the corresponding > Menu does not hide in the conference room menu. What are we missing after > the changes summarized below? What should we add/change in > RoomsPanel.java? Are we missing any line of code to be added in any > related file (Java or HTML or LZX)? From which Java method should we > update a Canvas attribute of LZX (if possible), or vice versa - refer to a > Java getter or attribute from an LZX constraint or method. > > Details: > > For the newly added "hideMathMenu" checkbox in RoomForm.java (parallel to > "hideActionsMenu"): > > add(new CheckBox("hideMathMenu")); > > We added the wicket-id section in RoomsPanel.html: > > <div class="formelement"> > > <label><wicket:message key="1681" /> </label> > > <input type="checkbox" > > class="formcheckbox" wicket:id="hideMathMenu"/> > > </div> > > There is no file called RoomForm.html or AdminBaseForm.html? Is there > another file that performs the similar function and needs to change? Is > any change needed in RoomsPanel.java ? > > We also updated all the related method definitions and calling lines in: > > RoomWebServiceFacade.java => RoomWebService.java => RoomManager.java => > Room.java (Entity) => RoomDTO.java. > > Sincerely, > > Susheel Jalali > Coscend Communications Solutions > India > Web site:www.Coscend.com > --------------------------------------------------------------- > CONFIDENTIALITY NOTICE: See 'Confidentiality Notice Regarding E-mail > Messages from Coscend Communications Solutions' posted at: > http://www.Coscend.com/Terms_and_Conditions.html > > > On 10/05/15 18:09, Maxim Solodovnik wrote: > >> you need to add additional attribute to the Room object >> and additional checkbox with correspondent wicket:id to the form for admin >> >> (both java and html file) >> >> On Sun, Oct 4, 2015 at 10:42 AM, Susheel Jalali < >> [email protected]> >> wrote: >> >> Dear Maxim, >>> >>> There is this additional information: >>> We also added a <div> in RoomsPanel.html for the new "hideMathMenu", >>> similar to "hideWhiteboard", and saw how ID 1439 to 1441 are used. >>> >>> Still the checkbox does not appear . We are missing some associated >>> changes to be made. >>> >>> Regards, >>> >>> Susheel Jalali >>> >>> Coscend Communications Solutions >>> >>> Elite Premio Complex Suite 200, Balewadi Pune 411045 Maharashtra India >>> >>> [email protected]_ >>> >>> Web site:www.Coscend.com <http://www.coscend.com/> >>> >>> ------------------------------------------------------------------ >>> >>> *Coscend’s**Software Service Factory* >>> >>> "*Coscend Communications* is ... *pioneering a new approach*to ... >>> software applications development, and systems integration." >>> >>> *Light Reading Network, *December, 2007 >>> >>> "*Coscend*is at the*vanguard of a new evolution*in telco OSS/BSS systems >>> integration." >>> >>> *Caroline Chappell* >>> A leading authority in the communications services software industry >>> >>> "There are *innovative*…*tools*from ... *Coscend *bubbling up, which will >>> help accelerate the data consolidation process and reduce its cost." >>> >>> *Dennis Mendyk, */Editor,/Building a *Telco Service Factory* >>> >>> ------------------------------------------------------------------ >>> >>> CONFIDENTIALITY NOTICE: See 'Confidentiality Notice Regarding E-mail >>> Messages from Coscend Communications Solutions' posted at: >>> http://www.Coscend.com/Terms_and_Conditions.html < >>> http://www.coscendcommunications.com/Terms_and_Conditions.html> >>> >>> >>> On 10/03/15 02:39, Susheel Jalali wrote: >>> >>> Dear Maxim and OpenMeetiings Developers, >>>> We would like to add a new Room parameter called "hideMathMenu" as a >>>> checkbox in ‘admin -> conf room -> layout’. It needs to behave similar >>>> to >>>> "hideActionsMenu", but for our own MathMenu. This will allow us to >>>> include >>>> this menu selectively in a few conference rooms. >>>> Below are details of how we have modified files that have >>>> "hideActionsMenu" and "hideWhiteboard". Still the checkbox does not >>>> appear >>>> in ADMIN => ROOM Configuration. Could you please advise us what we are >>>> missing? >>>> Thank you. ------------------- >>>> Detailed, relevant code >>>> We added a new variable mathmenuHidden in RoomDTO.java along with its >>>> getter/setter methods and initialization lines. >>>> private boolean mathmenuHidden; >>>> Then, Added column element for "hideMathmenu", along with getter/ setter >>>> in Room.java >>>> Next, Add new checkbox for "hasMathmenu" in RoomForm.java >>>> Do we need to add a status variable for hideActionsMenu similar to the >>>> one below? For example any XML file entry or Key need to be created, or >>>> there is a mention of "appendJavaScript("omRoomPanelInit();" in >>>> RoomForm.java and RoomsPanel.java, but we could not find it in the >>>> JavaScript. >>>> We could not use the Key 1439 to learn more, as it is no longer used. >>>> The >>>> code changes for OpenMeetings-20 to OpenMeetings-24 and OpenMEetings-89 >>>> issues are not accessible in JIRA. >>>> OpenmeetingsVariables.java >>>> Define static Boolean mathMenuStatus = null; // Similar to >>>> whiteboardDrawStatus >>>> Other changes that we did include: >>>> 1. ImportInitvalues.java >>>> r.setHideMathmenu( true ); >>>> // Near Line504, So that it does not appear unless added by >>>> Admin >>>> checkbox >>>> 2. conferenceMenubar.lzx >>>> In conferenceMenubar.lzx, defined new Menu entry with visible attribute: >>>> <baseMenuMeetingsItem text="$once{ canvas.getLabelName(1637) }" >>>> visible="${ >>>> (classroot.viewType=='interview') ? false : (( canvas.ismoderator ) ? >>>> (!canvas.currentRoomObj.hideMathmenu) : false ) }" >>>> 3. RoomManager.java >>>> To the "addRoom()" and "updateRoomInternal()" methods add a >>>> parameter after "hideWhiteboard": >>>> "hideMathmenu". Also added this parameter to all the places where >>>> these two methods are called, in this file or in the following two >>>> files. >>>> 4. RoomWebService.java >>>> 5. RoomWebServiceFacade.java >>>> >>>> Sincerely, >>>> >>>> Susheel Jalali >>>> >>>> Coscend Communications Solutions >>>> [email protected]_ >>>> >>>> Web site:www.Coscend.com >>>> >>>> ------------------------------------------------------------------ >>>> >>>> >>>> No virus found in this message. >>>> Checked by AVG - www.avg.com <http://www.avg.com> >>>> Version: 2015.0.6140 / Virus Database: 4435/10743 - Release Date: >>>> 10/02/15 >>>> >>>> >>>> >>>> >> > -- WBR Maxim aka solomax
