Checkbox

Checkbox incorrect wording on creating document test

Bug #821141 reported by ViscountStyx on 2011-08-04

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Checkbox	Fix Released	Low	Unassigned

Bug Description

Currently Checkbox asks the user to select "Create Document -> Empty File", which as of 11.10a2 is incorrect as the submenu now reads "Create Document -> Empty Document"

Related branches

lp:checkbox

lp:~roadmr/ubuntu/oneiric/checkbox/0.12.4

Merged into lp:ubuntu/oneiric/checkbox at revision 31

Mathieu Trudel-Lapierre: Approve on 2011-08-10

Ubuntu branches: Pending requested 2011-08-10

Diff: 264222 lines (+230443/-23395)

166 files modified

backend (+3/-0)
checkbox_gtk/gtk_interface.py (+21/-12)
data/settings/shutter.xml (+25/-0)
data/whitelists/default.whitelist (+120/-120)
debian/changelog (+50/-1)
debian/control (+1/-1)
debian/po/de.po (+119/-0)
debian/po/en_AU.po (+123/-0)
debian/po/en_GB.po (+123/-0)
debian/po/es.po (+120/-0)
debian/po/fr.po (+119/-0)
debian/po/he.po (+119/-0)
debian/po/hu.po (+121/-0)
debian/po/it.po (+119/-0)
debian/po/ja.po (+124/-0)
debian/po/nl.po (+133/-0)
debian/po/pl.po (+119/-0)
debian/po/pt_BR.po (+119/-0)
debian/po/ru.po (+119/-0)
debian/po/zh_TW.po (+121/-0)
gtk/checkbox.svg (+2478/-0)
jobs/audio.txt.in (+30/-19)
jobs/bluetooth.txt.in (+12/-12)
jobs/certification.txt.in (+0/-50)
jobs/cpu.txt.in (+5/-5)
jobs/disk.txt.in (+10/-10)
jobs/fingerprint.txt.in (+2/-2)
jobs/firewire.txt.in (+1/-1)
jobs/floppy.txt.in (+2/-2)
jobs/graphics-2d.txt.in (+0/-56)
jobs/graphics.txt.in (+143/-0)
jobs/hardware.txt.in (+0/-31)
jobs/info.txt.in (+125/-0)
jobs/input.txt.in (+16/-0)
jobs/install.txt.in (+1/-1)
jobs/kernel.txt.in (+0/-25)
jobs/keyboard.txt.in (+0/-72)
jobs/keys.txt.in (+72/-0)
jobs/kms.txt.in (+0/-13)
jobs/lid.txt.in (+0/-29)
jobs/local.txt.in (+92/-94)
jobs/media.txt.in (+0/-148)
jobs/mediacard.txt.in (+137/-0)
jobs/memory.txt.in (+11/-11)
jobs/miscellanea.txt.in (+37/-0)
jobs/monitor.txt.in (+43/-1)
jobs/netio.txt.in (+0/-16)
jobs/network.txt.in (+0/-134)
jobs/networking.txt.in (+144/-0)
jobs/optical.txt.in (+23/-12)
jobs/pcmcia-pcix.txt.in (+6/-0)
jobs/pcmcia.txt.in (+0/-6)
jobs/peripheral.txt.in (+0/-28)
jobs/power-management.txt.in (+305/-0)
jobs/power.txt.in (+0/-20)
jobs/shutdown.txt.in (+0/-9)
jobs/sleep.txt.in (+0/-202)
jobs/sru_suite.txt.in (+0/-226)
jobs/stress.txt.in (+26/-1)
jobs/unity.txt.in (+3/-8)
jobs/usb.txt.in (+35/-27)
jobs/user_apps.txt.in (+36/-36)
jobs/video.txt.in (+0/-123)
jobs/xorg.txt.in (+0/-18)
plugins/backend_info.py (+49/-13)
plugins/jobs_info.py (+6/-3)
plugins/jobs_prompt.py (+4/-0)
plugins/launchpad_report.py (+2/-2)
plugins/scripts_info.py (+3/-1)
po/POTFILES.in (+31/-5)
po/ace.po (+2613/-0)
po/af.po (+2613/-0)
po/am.po (+2619/-0)
po/ar.po (+2641/-532)
po/ast.po (+2926/-479)
po/be.po (+2630/-0)
po/bg.po (+2542/-516)
po/bn.po (+2822/-0)
po/bo.po (+2665/-0)
po/br.po (+2685/-0)
po/bs.po (+2911/-0)
po/ca.po (+2868/-540)
po/checkbox.pot (+1370/-381)
po/ckb.po (+2624/-0)
po/cs.po (+3003/-550)
po/cy.po (+2616/-0)
po/da.po (+2914/-463)
po/de.po (+3117/-583)
po/dv.po (+2610/-0)
po/el.po (+2792/-530)
po/en_AU.po (+3081/-0)
po/en_CA.po (+2420/-363)
po/en_GB.po (+3109/-527)
po/eo.po (+3156/-0)
po/es.po (+3098/-599)
po/et.po (+2652/-0)
po/eu.po (+2425/-384)
po/fa.po (+2610/-0)
po/fi.po (+2874/-522)
po/fr.po (+3019/-628)
po/gl.po (+3222/-0)
po/he.po (+2685/-512)
po/hi.po (+2702/-0)
po/hr.po (+2492/-418)
po/hu.po (+2942/-527)
po/hy.po (+2610/-0)
po/id.po (+2622/-511)
po/is.po (+2433/-321)
po/it.po (+2937/-561)
po/ja.po (+2582/-506)
po/jbo.po (+2277/-317)
po/ka.po (+2296/-336)
po/kk.po (+2644/-0)
po/km.po (+2610/-0)
po/kn.po (+2613/-0)
po/ko.po (+2408/-336)
po/ku.po (+2622/-0)
po/ky.po (+2610/-0)
po/lt.po (+2581/-529)
po/lv.po (+2444/-374)
po/mk.po (+2391/-430)
po/ml.po (+2613/-0)
po/mr.po (+2616/-0)
po/ms.po (+3103/-0)
po/my.po (+2635/-0)
po/nb.po (+2564/-526)
po/nds.po (+2277/-317)
po/ne.po (+2625/-0)
po/nl.po (+3050/-453)
po/nn.po (+2617/-0)
po/oc.po (+2911/-0)
po/pl.po (+2836/-520)
po/pt.po (+3022/-465)
po/pt_BR.po (+2978/-469)
po/ro.po (+2636/-455)
po/ru.po (+2977/-545)
po/si.po (+2613/-0)
po/sk.po (+2864/-561)
po/sl.po (+2850/-0)
po/sq.po (+2445/-398)
po/sr.po (+3069/-0)
po/sv.po (+2951/-588)
po/ta.po (+2616/-0)
po/te.po (+2610/-0)
po/th.po (+2645/-530)
po/tr.po (+2706/-419)
po/ug.po (+2602/-405)
po/uk.po (+2682/-380)
po/ur.po (+2610/-0)
po/vi.po (+2431/-344)
po/zh_CN.po (+2643/-498)
po/zh_HK.po (+2782/-0)
po/zh_TW.po (+2756/-531)
scripts/bluetooth_scan (+7/-0)
scripts/cdimage_resource (+2/-1)
scripts/connect_wireless (+44/-0)
scripts/keyboard_test (+9/-4)
scripts/network_bandwidth_test (+9/-15)
scripts/obex_send (+82/-0)
scripts/reconnect (+5/-0)
scripts/sleep_test (+14/-67)
scripts/usb_test (+141/-0)
scripts/wake_on_lan_test (+150/-0)
scripts/xrandr_cycle (+77/-3)
setup.cfg (+10/-18)
setup.py (+2/-2)

lp:ubuntu/oneiric/checkbox

Revision history for this message

ViscountStyx (laice) wrote on 2011-08-04:

user_apps.txt.in Edit (9.0 KiB, text/plain)

I have never made any modifications to an open source file before, so i have absolutely no idea if i'm doing the right thing here! please let me know if i'm misguided to do this or misguided in the way i have done it!

Attached is a patched checkbox/user_apps.txt.in with the correct wording.

Kind Regards

Revision history for this message

ViscountStyx (laice) wrote on 2011-08-04:

Ok so having researched "patches" a bit further it's not a patch, more a modified version of a complete file. Sorry if i'm being a bit nooby.

Daniel Manrique (roadmr) on 2011-08-04

Changed in checkbox:
status:	New → Confirmed
importance:	Undecided → Low

Revision history for this message

Daniel Manrique (roadmr) wrote on 2011-08-04:

Hi Andrew,

Thanks so much for reporting this bug and helping improve Checkbox and Ubuntu!

I have confirmed this bug report on an Oneiric box running Alpha 3. The troublesome text is still present in the latest development code for Checkbox.

The file you sent is fine in principle, however, since the code is under development, I see that a lot of the job descriptions have changed, thus, it's not possible to simply replace the current file with the one you sent.

So, since you mention that you did some research on producing patches, I was wondering if you'd like to take this as a learning opportunity. You could produce a patch that includes *only* the changed portion of the file using the diff command. Assume user_apps.txt.in.old is the original, flawed version and user_apps.txt.in.new is your fixed version. Then do this to generate a patch:

diff -u user_apps.txt.in.old user_apps.txt.in.new > user_apps.txt.in.patch

And attach the .patch file to the bug report. This will be easier for us to integrate into the Checkbox job files.

As an aside, if you'd like to learn more about how we collaborate developing Checkbox, among other open-source projects, the development code for checkbox is managed using the Bazaar version control tool. If you want to learn about Bazaar you can start here:

http://bazaar.canonical.com/en/

Thanks again for your contribution, please don't forget to let me know if producing/attaching the patch worked or if you have trouble doing it!

Regards,
- Daniel

Revision history for this message

ViscountStyx (laice) wrote on 2011-08-04:

user_apps.txt.in.patch Edit (644 bytes, text/plain)

Right well here goes my first ever attempt heh
Rather heartwarming to know that this would be released to most ubuntu desktop editions.

Patch attached.

I thank you for your courteous response and allowing me to develop my skills, i do hope i can use them to more benefit to the community that this little niggle of a patch!

Although probably better for one to start small like this :)

Let me know if everythings OK.

Kind Regards

Revision history for this message

Daniel Manrique (roadmr) wrote on 2011-08-05:

Hi Andrew,

Thanks so much for preparing the patch, it was a lot easier to integrate this way.

And thanks for your contribution! This change has been integrated into the latest checkbox development code and, barring unforeseen circumstances, will be included in Checkbox when 11.10 is released.