Joseph Boyer Jr. wrote: > Hi Mike, > > I will fix the trackback. > > I had originally written this for v1.0.3 and in that version if the system is > not found it will open a page to add the new system. This was main reason for > writing this, as I did not want to duplicate systems. > > As you mentioned, this is not a true search. May work on that as well. > > --joe > > Joseph Boyer Jr. > Liquidnet Holdings, Inc. > E [email protected] > T +1 646.660.8352 > C +1 646.284.8394 >
Excellent, very cool. There is a search function in api.py, so I suspect most of what needs to be done is to add it to remote.py (perhaps just for a few fields, and not arbitrary fields, to make things simple), and then add a page for it in the left hand menu. Maybe it could search multiple object types from the same page? (This would require a new function in CobblerWeb.py also, so that we could call that function and get back the results). I can definitely see this being useful for other XMLRPC consumers as well. Thanks again, --Michael > > -----Original Message----- > From: [email protected] > [mailto:[email protected]] On Behalf Of Michael DeHaan > Sent: Monday, January 26, 2009 2:42 PM > To: cobbler mailing list > Subject: Re: PATCH: paginate.tmpl > > Joseph Boyer Jr. wrote: > >> Folks, >> >> >> >> I have a many systems in my cobbler environment, over 500 as of now, >> and it became a real pain to sift through the dozen pages of so >> looking for a particular machine. Thus, I have added a search field >> and number of items to display field to the default paginate.tmpl >> file. I find it useful and though other that use the webUI may as well. >> >> >> >> Cheers, >> >> joe >> >> >> >> Joseph Boyer Jr.* * >> *Liquidnet Holdings, Inc. * >> >> *E** *[email protected] <mailto:[email protected]> >> *T* +1 646.660.8352 >> >> *C* +1 646.284.8394 >> >> >> >> ------------------------------------------------------------------------ >> >> _______________________________________________ >> cobbler mailing list >> [email protected] >> https://fedorahosted.org/mailman/listinfo/cobbler >> >> > > I applied this to devel to try it out as a "search", I'm seeing a > traceback .... > > > > MOD_PYTHON ERROR > > ProcessId: 23323 > Interpreter: 'localhost.localdomain' > > ServerName: 'localhost.localdomain' > DocumentRoot: '/var/www/html' > > URI: '/cobbler/web/' > Location: None > Directory: '/var/www/cobbler/web/' > Filename: '/var/www/cobbler/web/' > PathInfo: '' > > Phase: 'PythonHandler' > Handler: 'index' > > Traceback (most recent call last): > > File "/usr/lib/python2.5/site-packages/mod_python/importer.py", line 1537, > in HandlerDispatch > default=default_handler, arg=req, silent=hlist.silent) > > File "/usr/lib/python2.5/site-packages/mod_python/importer.py", line 1229, > in _process_target > result = _execute_target(config, req, object, arg) > > File "/usr/lib/python2.5/site-packages/mod_python/importer.py", line 1128, > in _execute_target > result = object(arg) > > File "/var/www/cobbler/web/index.py", line 130, in handler > content = func( **form ) > > File "/usr/lib/python2.5/site-packages/cobbler/webui/CobblerWeb.py", line > 463, in system_edit > 'profiles': self.remote.get_profiles() > > File "/usr/lib/python2.5/site-packages/cobbler/webui/CobblerWeb.py", line > 102, in __render > return str(tmpl) > > File "/usr/lib/python2.5/site-packages/Cheetah/Template.py", line 982, in > __str__ > def __str__(self): return getattr(self, mainMethName)() > > File "/usr/lib/python2.5/site-packages/cobbler/webui/master.py", line 247, > in respond > self.body(trans=trans) > > File "_usr_share_cobbler_webui_templates_system_edit_tmpl.py", line 550, in > body > > NotFound: cannot find 'name' while searching for 'system.name' > > > MODULE CACHE DETAILS > > Accessed: Mon Jan 26 14:35:21 2009 > Generation: 0 > > _mp_43be29589f5d909275df7d4c30cbcdd3 { > FileName: '/var/www/cobbler/web/index.py' > Instance: 1 [IMPORT] > Generation: 1 > Modified: Mon Jan 26 14:32:46 2009 > Imported: Mon Jan 26 14:35:21 2009 > > > } > > > I had 1000 systems in the list, named autotest-0 through autotest-1000 > (generated by tests/performance.py). What I typed in was "10", not > "autotest-10". > > From the code I see, I take it what you want to do is not really > "search", but rather a "open the page specifically for this system named > ____", in which case it's not a search, is it? I can see both being > valuable. This is still good. > > If you can add some error handling that detects the page not found error > a little better and perhaps rename "Search" to something like "System > Name:" (I can't think of anything better but there HAS to be a word for > it), I'll apply this. > > Ideally we'd also get search going in the XMLRPC and make it a lot > smarter, at which point we can come up with a new search page on top of > this and still leave this in. > > Thanks! > > --Michael > > > _______________________________________________ > cobbler mailing list > [email protected] > https://fedorahosted.org/mailman/listinfo/cobbler > > > _______________________________________________ > cobbler mailing list > [email protected] > https://fedorahosted.org/mailman/listinfo/cobbler > _______________________________________________ cobbler mailing list [email protected] https://fedorahosted.org/mailman/listinfo/cobbler
